-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[Needs payment][Track Tax] Add new input fields for tax tracking in Money requests #31672
Comments
Triggered auto assignment to @kadiealexander ( |
Hi, I'm Eto from Callstack - expert contributor group - and I would like to take care of this issue. |
Hey, Im Artem from Callstack, and would like to help with this issue |
We're discussing next steps in slack. |
@teneeto, @MonilBhavsar, @waterim, @kadiealexander Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Hi @MonilBhavsar Couldn't raise a draft earlier. But I Should today. |
Thank you @teneeto ! |
Issue is already updated, Melvin! |
@teneeto, @MonilBhavsar, @waterim, @kadiealexander Still overdue 6 days?! Let's take care of this! |
Not overdue! |
Draft PR is up! |
@alitoshmatov the second PR is ready for review! |
This issue has not been updated in over 15 days. @teneeto, @MonilBhavsar, @waterim, @kadiealexander, @alitoshmatov eroding to Monthly issue. P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do! |
Is there any plan for our next step on this feature? |
We're working on other parallel issues. We can issue payment here and close this now |
@kadiealexander Can we complete payment and close this one |
Job added to Upwork: https://www.upwork.com/jobs/~01fe3f2f6f3d94b969 |
Current assignee @alitoshmatov is eligible for the Internal assigner, not assigning anyone new. |
@MonilBhavsar I'm finding this hard to follow, could you please quickly summarise who is due payment for what? |
Payment summary: Pay $1000 to @alitoshmatov for reviewing two PR's created before the price update There were some regressions coming out of it, but since it was a new feature and lots of new introductions to App, i think it's fine to pay full payment. |
Eto Olei is from Callstack and does not need payment |
@kadiealexander Bump |
@alitoshmatov I've sent you an upwork contract. |
@kadiealexander Accepted the offer |
Payouts due:
Upwork job is here. |
Problem
Internal master Issue:https://github.com/Expensify/Expensify/issues/338212
Coming from this section in design doc https://docs.google.com/document/d/1nkusVPiYN3uYCJAH_2XknBv_oPpNOifVxheuxRvFTxs/edit?pli=1#bookmark=id.l319wrhremwa
This issue is divided in 2 phases to keep PR's small, test use cases efficiently and deliver in milestones
Phase 1
In phase 1, we create a simple input fields that can allow inserting, editing and viewing tax details
As discussed in design doc, we need to create tax selector and tax amount components. Also add handles to update transaction tax details in Onyx and also pass tax details to API
Components and handlers should be created in a new way following this refactor PR #28618
Phase 2
In phase 2, we'll add more logic to tax input fields.
Why is this important?
To allow users to track tax on the expenses, and migration users to newdot.
Solution
Implement the section of the design doc
Issue Owner
Current Issue Owner: @kadiealexanderUpwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: