Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[$250] [HOLD for payment] [Details Revamp] Allow Editing Report Title from Details Page #44149

Closed
grgia opened this issue Jun 21, 2024 · 24 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors NewFeature Something to build that is a new item. Weekly KSv2

Comments

@grgia
Copy link
Contributor

grgia commented Jun 21, 2024

Problem

image

We currently display the report title as shown above. This is now inconsistent with our new details revamp project.

Solution

image

Make the report title editor available from the details page for those with correct permissions. Remove it from the report body as shown in the previous screenshot.

So for example, members, approvers and admins can change the report title. Anyone added to the room, not part of the approval workfllow, should not be able to change the report title.

We will use our typical push to page pattern for this change.

Slack thread

https://expensify.slack.com/archives/C036QM0SLJK/p1718830874704809?thread_ts=1718719187.256049&cid=C036QM0SLJK

Issue OwnerCurrent Issue Owner: @
Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01cc27f37e79795306
  • Upwork Job ID: 1813753312852314259
  • Last Price Increase: 2024-07-18
Issue OwnerCurrent Issue Owner: @ishpaul777
@grgia grgia added Daily KSv2 NewFeature Something to build that is a new item. labels Jun 21, 2024
@grgia grgia self-assigned this Jun 21, 2024
Copy link

melvin-bot bot commented Jun 21, 2024

Triggered auto assignment to @kadiealexander (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details. Please add this Feature request to a GH project, as outlined in the SO.

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Daily KSv2 labels Jun 21, 2024
Copy link

melvin-bot bot commented Jun 21, 2024

⚠️ It looks like this issue is labelled as a New Feature but not tied to any GitHub Project. Keep in mind that all new features should be tied to GitHub Projects in order to properly track external CAP software time ⚠️

Copy link

melvin-bot bot commented Jun 21, 2024

Triggered auto assignment to Design team member for new feature review - @dubielzyk-expensify (NewFeature)

@cdOut
Copy link
Contributor

cdOut commented Jun 21, 2024

Hi, I’m Tymoteusz from Software Mansion, the expert agency, and I’ll be working on this issue.

@ishpaul777
Copy link
Contributor

I am low on issues, just incase we need a c+ for this issue i'd love to help

@trjExpensify
Copy link
Contributor

Assigning @ishpaul777 for C+.

Copy link

melvin-bot bot commented Jul 8, 2024

⚠️ Looks like this issue was linked to a Deploy Blocker here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

Copy link

melvin-bot bot commented Jul 8, 2024

⚠️ Looks like this issue was linked to a Deploy Blocker here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

@melvin-bot melvin-bot bot added Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Weekly KSv2 labels Jul 10, 2024
@melvin-bot melvin-bot bot changed the title [Details Revamp] Allow Editing Report Title from Details Page [HOLD for payment 2024-07-17] [Details Revamp] Allow Editing Report Title from Details Page Jul 10, 2024
@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Jul 10, 2024
@melvin-bot melvin-bot bot added the Daily KSv2 label Jul 16, 2024
@kadiealexander kadiealexander changed the title [HOLD for payment 2024-07-22] [HOLD for payment 2024-07-17] [Details Revamp] Allow Editing Report Title from Details Page [HOLD for payment 2024-07-17] [Details Revamp] Allow Editing Report Title from Details Page Jul 17, 2024
Copy link

melvin-bot bot commented Jul 17, 2024

Payment Summary

Upwork Job

  • Contributor: @cdOut is from an agency-contributor and not due payment
  • Reviewer: @ishpaul777 paid $250 via Upwork (offer)

BugZero Checklist (@kadiealexander)

  • I have verified the correct assignees and roles are listed above and updated the neccesary manual offers
  • I have verified that there are no duplicate or incorrect contracts on Upwork for this job (https://www.upwork.com/ab/applicants//hired)
  • I have paid out the Upwork contracts or cancelled the ones that are incorrect
  • I have verified the payment summary above is correct

@kadiealexander kadiealexander added the External Added to denote the issue can be worked on by a contributor label Jul 18, 2024
Copy link

melvin-bot bot commented Jul 18, 2024

Job added to Upwork: https://www.upwork.com/jobs/~01cc27f37e79795306

@melvin-bot melvin-bot bot changed the title [HOLD for payment 2024-07-17] [Details Revamp] Allow Editing Report Title from Details Page [$250] [HOLD for payment 2024-07-17] [Details Revamp] Allow Editing Report Title from Details Page Jul 18, 2024
@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Jul 18, 2024
Copy link

melvin-bot bot commented Jul 18, 2024

Current assignee @ishpaul777 is eligible for the External assigner, not assigning anyone new.

@kadiealexander
Copy link
Contributor

@ishpaul777 please propose the regression test steps here.

Copy link

melvin-bot bot commented Jul 23, 2024

@grgia, @kadiealexander, @cdOut, @ishpaul777, @dubielzyk-expensify Huh... This is 4 days overdue. Who can take care of this?

@melvin-bot melvin-bot bot added the Overdue label Jul 23, 2024
@ishpaul777
Copy link
Contributor

ishpaul777 commented Jul 23, 2024

Regression Test proposal:

  1. Create a workspace.
  2. Submit two separate expenses for said workspace.
  3. Go into the money report view of those workspace expenses.
  4. Click on the report header.
  5. Verify the report title field is editable.
Screenshot 2024-07-24 at 12 10 51 AM
  1. Verify only members, approvers and admins of workspace be able to edit it, report invitees should not have that option

Do we agree 👍 or 👎

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Jul 23, 2024
@kadiealexander
Copy link
Contributor

@ishpaul777 please accept the upwork offer.

@melvin-bot melvin-bot bot removed the Overdue label Jul 26, 2024
@ishpaul777
Copy link
Contributor

Hey @kadiealexander Would you please hold my payment for few days, i have had my payments on HOLD across issues i'll bump once i am ready to accept payment.

Reason: #44029 (comment)

@kadiealexander kadiealexander changed the title [$250] [HOLD for payment 2024-07-17] [Details Revamp] Allow Editing Report Title from Details Page [$250] [HOLD for payment] [Details Revamp] Allow Editing Report Title from Details Page Jul 28, 2024
@melvin-bot melvin-bot bot added the Overdue label Jul 28, 2024
@kadiealexander
Copy link
Contributor

No worries!

@melvin-bot melvin-bot bot removed the Overdue label Jul 28, 2024
@kadiealexander kadiealexander added Weekly KSv2 and removed Daily KSv2 labels Jul 28, 2024
@ishpaul777
Copy link
Contributor

Still hold

@ishpaul777
Copy link
Contributor

Thanks for patience here @kadiealexander, please release payment when you get the chance

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Sep 3, 2024
@github-project-automation github-project-automation bot moved this from Release 2: Summer 2024 (Aug) to Done in [#whatsnext] #wave-collect Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors NewFeature Something to build that is a new item. Weekly KSv2
Projects
No open projects
Status: Done
Development

No branches or pull requests

6 participants