-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Comments
Triggered auto assignment to @jliexpensify ( |
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 |
Off hold now that the other issue is on prod. |
@luacmartins Could I take over this one? |
@DylanDylann you mean as a C+ |
@mountiny Yeah, because I have reviewed other pages of tax feature |
Thats reasonable, assign you |
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? |
Sounds good |
|
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:
|
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:
|
There were 4 pages which were fixed in here, I suggest paying $500 to @DylanDylann for the review cc @luacmartins @trjExpensify of #38208 |
Sounds good to me! |
Job added to Upwork: https://www.upwork.com/jobs/~011e446073ea5b5a4e |
Payment Summary
|
Current assignee @DylanDylann is eligible for the External assigner, not assigning anyone new. |
@DylanDylann have invited you here |
Waiting on @DylanDylann to accept. Also, any Regression Tests needed here? |
No regression tests as part of this issue, we're adding it as part of the project instead. |
Paid and job closed! |
Implement this section of the Simplified Collect: Taxes doc
cc @kosmydel @filip-solecki
Issue Owner
Current Issue Owner: @jliexpensifyUpwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: