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

[$500] [HOLD for payment 2024-04-03] [Simplified Collect][Taxes] Create WorkspaceEditTaxPage #37794

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

Comments

@luacmartins
Copy link
Contributor

luacmartins commented Mar 5, 2024

Implement this section of the Simplified Collect: Taxes doc

cc @kosmydel @filip-solecki

Issue OwnerCurrent Issue Owner: @jliexpensify
Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~011e446073ea5b5a4e
  • Upwork Job ID: 1775735005542027264
  • Last Price Increase: 2024-04-04
@luacmartins luacmartins added Daily KSv2 NewFeature Something to build that is a new item. labels Mar 5, 2024
@luacmartins luacmartins self-assigned this Mar 5, 2024
Copy link

melvin-bot bot commented Mar 5, 2024

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Daily KSv2 labels Mar 5, 2024
@luacmartins luacmartins moved this to Release 1: Spring 2024 (May) in [#whatsnext] #wave-collect Mar 5, 2024
@luacmartins luacmartins changed the title [Simplified Collect][Taxes] Create WorkspaceEditTaxPage [HOLD #37787][Simplified Collect][Taxes] Create WorkspaceEditTaxPage Mar 5, 2024
@kosmydel
Copy link
Contributor

kosmydel commented Mar 13, 2024

Hey, I will take care of this issue.

I've prepared a draft PR of it. I will continue to work on this when the SetPolicyTaxesEnabled command is deployed on the backend.

@mountiny mountiny self-assigned this Mar 14, 2024
@garrettmknight garrettmknight changed the title [HOLD #37787][Simplified Collect][Taxes] Create WorkspaceEditTaxPage [Simplified Collect][Taxes] Create WorkspaceEditTaxPage Mar 15, 2024
@garrettmknight
Copy link
Contributor

Off hold now that the other issue is on prod.

@DylanDylann
Copy link
Contributor

@luacmartins Could I take over this one?

@mountiny
Copy link
Contributor

@DylanDylann you mean as a C+

@DylanDylann
Copy link
Contributor

DylanDylann commented Mar 18, 2024

@mountiny Yeah, because I have reviewed other pages of tax feature

@mountiny
Copy link
Contributor

Thats reasonable, assign you

@jliexpensify
Copy link
Contributor

Hello, I am OOO from 21st to 31st March. I won't assign this to another B0 member, as I think during this period, there are likely no actions for me to take, right?

@mountiny
Copy link
Contributor

Sounds good

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Weekly KSv2 labels Mar 19, 2024
@melvin-bot melvin-bot bot removed the Weekly KSv2 label Mar 27, 2024
@melvin-bot melvin-bot bot added Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production labels Mar 27, 2024
@melvin-bot melvin-bot bot changed the title [Simplified Collect][Taxes] Create WorkspaceEditTaxPage [HOLD for payment 2024-04-03] [Simplified Collect][Taxes] Create WorkspaceEditTaxPage Mar 27, 2024
Copy link

melvin-bot bot commented Mar 27, 2024

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

@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Mar 27, 2024
Copy link

melvin-bot bot commented Mar 27, 2024

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.56-8 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-04-03. 🎊

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

  • @kosmydel does not require payment (Contractor)
  • @DylanDylann requires payment (Needs manual offer from BZ)

Copy link

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

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

@mountiny
Copy link
Contributor

There were 4 pages which were fixed in here, I suggest paying $500 to @DylanDylann for the review cc @luacmartins @trjExpensify of #38208

@trjExpensify
Copy link
Contributor

Sounds good to me!

@jliexpensify jliexpensify added the External Added to denote the issue can be worked on by a contributor label Apr 4, 2024
@melvin-bot melvin-bot bot changed the title [HOLD for payment 2024-04-03] [Simplified Collect][Taxes] Create WorkspaceEditTaxPage [$500] [HOLD for payment 2024-04-03] [Simplified Collect][Taxes] Create WorkspaceEditTaxPage Apr 4, 2024
Copy link

melvin-bot bot commented Apr 4, 2024

Job added to Upwork: https://www.upwork.com/jobs/~011e446073ea5b5a4e

@jliexpensify
Copy link
Contributor

Payment Summary

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

melvin-bot bot commented Apr 4, 2024

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

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels Apr 4, 2024
@jliexpensify jliexpensify removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Apr 4, 2024
@jliexpensify
Copy link
Contributor

@DylanDylann have invited you here

@jliexpensify
Copy link
Contributor

Waiting on @DylanDylann to accept. Also, any Regression Tests needed here?

@luacmartins
Copy link
Contributor Author

No regression tests as part of this issue, we're adding it as part of the project instead.

@jliexpensify
Copy link
Contributor

Paid and job closed!

@github-project-automation github-project-automation bot moved this from Release 1: Spring 2024 (May) to Done in [#whatsnext] #wave-collect Apr 9, 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 Daily KSv2 External Added to denote the issue can be worked on by a contributor NewFeature Something to build that is a new item.
Projects
No open projects
Development

No branches or pull requests

7 participants