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-29] [$500][Simplified Collect][Tags] Implement DeleteWorkspaceTags #37314

Closed
luacmartins opened this issue Feb 27, 2024 · 23 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 NewFeature Something to build that is a new item. Weekly KSv2

Comments

@luacmartins
Copy link
Contributor

luacmartins commented Feb 27, 2024

Implement this section of the Simplified Collect: Tags design doc

cc @waterim

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~0155b70726d736408f
  • Upwork Job ID: 1762500850869534720
  • Last Price Increase: 2024-02-27
@luacmartins luacmartins added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Feb 27, 2024
@luacmartins luacmartins self-assigned this Feb 27, 2024
@luacmartins luacmartins moved this to Release 2: From Setup > Export in [#whatsnext] Wave 08 - Collect Plan Admins Feb 27, 2024
Copy link

melvin-bot bot commented Feb 27, 2024

Triggered auto assignment to @MitchExpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@waterim
Copy link
Contributor

waterim commented Feb 27, 2024

Hello, Im Artem from Callstack and would like to take this one

@luacmartins luacmartins added External Added to denote the issue can be worked on by a contributor and removed Bug Something is broken. Auto assigns a BugZero manager. labels Feb 27, 2024
@melvin-bot melvin-bot bot changed the title [HOLD #37309][Simplified Collect][Tags] Implement DeleteWorkspaceTags [$500] [HOLD #37309][Simplified Collect][Tags] Implement DeleteWorkspaceTags Feb 27, 2024
Copy link

melvin-bot bot commented Feb 27, 2024

Job added to Upwork: https://www.upwork.com/jobs/~0155b70726d736408f

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Feb 27, 2024
@luacmartins luacmartins added NewFeature Something to build that is a new item. and removed Help Wanted Apply this label when an issue is open to proposals by contributors labels Feb 27, 2024
Copy link

melvin-bot bot commented Feb 27, 2024

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

Copy link

melvin-bot bot commented Feb 27, 2024

Current assignee @MitchExpensify is eligible for the NewFeature assigner, not assigning anyone new.

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Daily KSv2 labels Feb 27, 2024
@luacmartins luacmartins added Daily KSv2 and removed Weekly KSv2 labels Feb 27, 2024
@melvin-bot melvin-bot bot added the Overdue label Feb 29, 2024
@luacmartins
Copy link
Contributor Author

Still on hold for the main page

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Mar 1, 2024
@MitchExpensify
Copy link
Contributor

PR was merged so I'm taking this off hold!

@melvin-bot melvin-bot bot removed the Overdue label Mar 5, 2024
@MitchExpensify
Copy link
Contributor

Are we good to apply Help Wanted on this now @luacmartins ?

@melvin-bot melvin-bot bot added the Overdue label Mar 14, 2024
@luacmartins
Copy link
Contributor Author

@waterim I've asked @allroundexperts to get started on implementing the delete functionality while you finish settings and edit pages. Hope that's ok!

@melvin-bot melvin-bot bot removed the Overdue label Mar 15, 2024
@allroundexperts
Copy link
Contributor

allroundexperts commented Mar 17, 2024

Draft PR created #38453

This has some conflicts because the PR it is based from has conflicts as well.

I will make it it ready for review once the base PR gets merged.

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

allroundexperts commented Mar 17, 2024

Okay. I was mistaken. The main branch already had the tag setting page created. As such, we don't need to base off from the above branch. The PR is ready for review.

@melvin-bot melvin-bot bot added the Overdue label Mar 18, 2024
@melvin-bot melvin-bot bot added Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Weekly KSv2 Overdue labels Mar 19, 2024
@melvin-bot melvin-bot bot changed the title [$500][Simplified Collect][Tags] Implement DeleteWorkspaceTags [HOLD for payment 2024-03-29] [$500][Simplified Collect][Tags] Implement DeleteWorkspaceTags Mar 22, 2024
@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Mar 22, 2024
Copy link

melvin-bot bot commented Mar 22, 2024

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

Copy link

melvin-bot bot commented Mar 22, 2024

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.55-3 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-29. 🎊

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

Copy link

melvin-bot bot commented Mar 22, 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:

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

@MitchExpensify
Copy link
Contributor

Reminder set to pay.

Payment summary:

C+ $500 @allroundexperts (NewDot)

@github-project-automation github-project-automation bot moved this from Release 1: Spring 2024 (May) to Done in [#whatsnext] #wave-collect Mar 25, 2024
@JmillsExpensify
Copy link

$500 approved for @allroundexperts

@Ollyws
Copy link
Contributor

Ollyws commented Apr 10, 2024

@MitchExpensify I just noticed I was never paid for my review of #38453, any chance you could help with that? Thanks!

@MitchExpensify
Copy link
Contributor

Can do, Payment job here https://www.upwork.com/jobs/~011908d15db493b1bf @Ollyws

@Ollyws
Copy link
Contributor

Ollyws commented Apr 11, 2024

@MitchExpensify Accepted, thanks!

@MitchExpensify
Copy link
Contributor

Done!

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 NewFeature Something to build that is a new item. Weekly KSv2
Projects
No open projects
Archived in project
Status: Release 2: From Setup > Export
Development

No branches or pull requests

6 participants