-
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
[$250] [HOLD for payment 2025-02-17] [CRITICAL] Add dropdown to allow users to choose which domain their travel should be provisioned under in NewDot #55377
Comments
Original issue: https://github.com/Expensify/Expensify/issues/445940 |
I can help with this ! |
@dannymcclain, @cristipaval, @ZhenjaHorbach Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Not overdue |
Backend PRs are deployed. App PR passed the design review and it is being reviewed by the C+ |
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. |
@tgolen, @garrettmknight, @dannymcclain, @cristipaval, @ZhenjaHorbach Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Looks like that PR went to prod. @ZhenjaHorbach any others expected? |
I hope that's all 😄 |
Cool, we'll wait till payout then! @cristipaval do we need a regression test for this one or is it being handled elsewhere? |
Good question! I think we'll add regression tests when we enable travel for everyone, right now Applause can't really test this. @stitesExpensify what do you think? |
Payment Summary
BugZero Checklist (@garrettmknight)
|
@garrettmknight I think we're good to issue the payment for @ZhenjaHorbach here |
Job added to Upwork: https://www.upwork.com/jobs/~021891556443234572502 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @brunovjk ( |
📣 @ZhenjaHorbach 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
All paid up! |
Problem
Currently when we provision travel, we use the domain of the billing owner for that policy. This works fine in most cases, but if there is an external accountant set as the owner, then the policy would be provisioned under that company which is incorrect and would need to be manually fixed by us.
Solution
When enabling travel, display a dropdown that includes all domains of members on the policy with a question like "Which company does this workspace belong to?"This will then be passed to the backend and placed underneath that spotnana companyAfter this long thread, we landed on the following solution:
(recommended)
in front of the most used domain in that policyUpwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @brunovjkThe text was updated successfully, but these errors were encountered: