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

[HOLD for payment 2024-03-22] [$500] [Simplified Collect][Categories] CategorySettingsPage: rename category, delete category, onyx actions #35709

Closed
luacmartins opened this issue Feb 2, 2024 · 32 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Engineering External Added to denote the issue can be worked on by a contributor NewFeature Something to build that is a new item. Weekly KSv2

Comments

@luacmartins
Copy link
Contributor

luacmartins commented Feb 2, 2024

Implement CategorySettingsPage: rename category, delete category, onyx actions from the CategorySettingsPage: page init, enable/disable categories, display name, onyx action for enabling/disabling category

Issue OwnerCurrent Issue Owner: @ArekChr
Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~019c879178a6d4badc
  • Upwork Job ID: 1755658765113217024
  • Last Price Increase: 2024-02-08
@luacmartins luacmartins added Engineering Daily KSv2 NewFeature Something to build that is a new item. labels Feb 2, 2024
@luacmartins luacmartins self-assigned this Feb 2, 2024
Copy link

melvin-bot bot commented Feb 2, 2024

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Daily KSv2 labels Feb 2, 2024
@luacmartins
Copy link
Contributor Author

@ArekChr can you please comment here so I can assign you to the issue?

@luacmartins luacmartins added Daily KSv2 and removed Weekly KSv2 labels Feb 2, 2024
@luacmartins luacmartins changed the title CategorySettingsPage: rename category, delete category, onyx actions [Simplified Collect] [Categories] CategorySettingsPage: rename category, delete category, onyx actions Feb 2, 2024
@luacmartins luacmartins changed the title [Simplified Collect] [Categories] CategorySettingsPage: rename category, delete category, onyx actions [Simplified Collect][Categories] CategorySettingsPage: rename category, delete category, onyx actions Feb 2, 2024
@melvin-bot melvin-bot bot added the Overdue label Feb 5, 2024
@ArekChr
Copy link
Contributor

ArekChr commented Feb 5, 2024

Sure, please assign me @luacmartins

Copy link

melvin-bot bot commented Feb 8, 2024

@slafortune, @luacmartins, @ArekChr Whoops! This issue is 2 days overdue. Let's get this updated quick!

@melvin-bot melvin-bot bot added the Overdue label Feb 8, 2024
@luacmartins luacmartins added the External Added to denote the issue can be worked on by a contributor label Feb 8, 2024
@melvin-bot melvin-bot bot changed the title [Simplified Collect][Categories] CategorySettingsPage: rename category, delete category, onyx actions [$500] [Simplified Collect][Categories] CategorySettingsPage: rename category, delete category, onyx actions Feb 8, 2024
Copy link

melvin-bot bot commented Feb 8, 2024

Job added to Upwork: https://www.upwork.com/jobs/~019c879178a6d4badc

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Feb 8, 2024
Copy link

melvin-bot bot commented Feb 8, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @akinwale (External)

@melvin-bot melvin-bot bot removed the Overdue label Feb 8, 2024
@luacmartins luacmartins removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Feb 8, 2024
@jeremy-croff
Copy link
Contributor

Hi I'm trying to review this issue but do not have access to the document in the OP.
@luacmartins

Copy link

melvin-bot bot commented Feb 12, 2024

@akinwale, @slafortune, @luacmartins, @ArekChr Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@melvin-bot melvin-bot bot added the Overdue label Feb 12, 2024
@melvin-bot melvin-bot bot added the Overdue label Feb 28, 2024
@ArekChr
Copy link
Contributor

ArekChr commented Feb 29, 2024

Hey, the main category settings page is currently under review, which is needed for this feature. It will soon be merged, and then I'll create a draft PR

@melvin-bot melvin-bot bot removed the Overdue label Feb 29, 2024
@slafortune
Copy link
Contributor

Great! Thanks for the details!

@luacmartins luacmartins changed the title [Hold #35705][$500] [Simplified Collect][Categories] CategorySettingsPage: rename category, delete category, onyx actions [$500] [Simplified Collect][Categories] CategorySettingsPage: rename category, delete category, onyx actions Mar 1, 2024
@ArekChr
Copy link
Contributor

ArekChr commented Mar 4, 2024

Hey there, soon I will create draft PR

@luacmartins
Copy link
Contributor Author

@ArekChr do you have a draft PR to link to this issue yet?

@ArekChr
Copy link
Contributor

ArekChr commented Mar 5, 2024

@luacmartins Linked draft PR

@luacmartins
Copy link
Contributor Author

Thank you!

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Weekly KSv2 labels Mar 6, 2024
@ArekChr
Copy link
Contributor

ArekChr commented Mar 6, 2024

PR ready for review

Copy link

melvin-bot bot commented Mar 14, 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 Mar 15, 2024
@melvin-bot melvin-bot bot changed the title [$500] [Simplified Collect][Categories] CategorySettingsPage: rename category, delete category, onyx actions [HOLD for payment 2024-03-22] [$500] [Simplified Collect][Categories] CategorySettingsPage: rename category, delete category, onyx actions Mar 15, 2024
@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Mar 15, 2024
Copy link

melvin-bot bot commented Mar 15, 2024

Reviewing label has been removed, please complete the "BugZero Checklist".

Copy link

melvin-bot bot commented Mar 15, 2024

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.52-6 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

If no regressions arise, payment will be issued on 2024-03-22. 🎊

For reference, here are some details about the assignees on this issue:

  • @akinwale requires payment (Needs manual offer from BZ)
  • @ArekChr does not require payment (Contractor)

Copy link

melvin-bot bot commented Mar 15, 2024

BugZero Checklist: The PR adding this new feature has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

  • [@akinwale] Please propose regression test steps to ensure the new feature will work correctly on production in further releases.
  • [@slafortune] Link the GH issue for creating/updating the regression test once above steps have been agreed upon.

@slafortune
Copy link
Contributor

@akinwale can you propose regression test steps to ensure the new feature will work correctly on production in further releases so I can get this paid today?

@akinwale
Copy link
Contributor

  • [@akinwale] Please propose regression test steps to ensure the new feature will work correctly on production in further releases.

Regression Test Steps

  • Launch Expensify
  • Log in as an the admin of a paid policy (Collect or Control) and navigate to the Workspace Settings > Select a workspace > Categories.
  • Choose some categories that are currently disabled.
  • Click the options menu in the top right corner and select "Enable Categories".
  • Verify that the selected categories are now enabled.
  • Choose some categories that are currently enabled.
  • Click the options menu in the top right corner again and select "Disable Categories".
  • Verify that these categories are now disabled.
  • Choose any categories you wish to delete.
  • In the top right corner, click the options menu and choose "Delete Categories".
  • Verify that the selected categories were removed.

Do we agree 👍 or 👎?

@akinwale
Copy link
Contributor

@slafortune Done!

@slafortune
Copy link
Contributor

@akinwale
Copy link
Contributor

Offer sent @akinwale https://www.upwork.com/nx/wm/offer/101522153

Accepted. Thanks!

@slafortune
Copy link
Contributor

@akinwale - Paid $500 via Upworks
@ArekChr does not require payment (Contractor)

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 Engineering External Added to denote the issue can be worked on by a contributor NewFeature Something to build that is a new item. Weekly KSv2
Projects
No open projects
Archived in project
Status: Release 1: Ideal Nav & Collect Simplfied Profile, Members, Categories, Workflows (approvals)
Development

No branches or pull requests

6 participants