-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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] [OldDot Rules Migration] Expense report rules #47014
Comments
Hey, I'm from SWM, I'd like to take over this issue, would you please assign me? |
Assigned! |
Triggered auto assignment to @strepanier03 ( |
@marcaaron, @BrtqKr Eep! 4 days overdue now. Issues have feelings too... |
@BrtqKr when do you expect to make progress on this one? |
@garrettmknight, the functionality in the PR is ready, we're waiting to merge the PR with enabling expense rules for the workspace before both sections to limit the diff. After that I'll record the videos and pass it further |
What's your email? |
Ah sorry. My email: dylandylan9112@gmail.com |
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. |
This issue has not been updated in over 15 days. @marcaaron, @BrtqKr 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! |
@marcaaron This is ready for payment. Could you please assign a BZ to handle payment? 🙏 |
Triggered auto assignment to @VictoriaExpensify ( |
@VictoriaExpensify can you help pay @DylanDylann and then close this out after? It's done and just waiting for payment. |
Job added to Upwork: https://www.upwork.com/jobs/~021844160179253102195 |
Current assignee @DylanDylann is eligible for the External assigner, not assigning anyone new. |
Hey @DylanDylann - I've sent you a job offer, can you please accept so I can pay you? https://www.upwork.com/nx/wm/offer/104347866 |
@VictoriaExpensify Sure I did accept |
Contributor: @BrtqKr - No payment required, Contractor |
Part of the OldDot Rules Migration project
Main issue: https://github.com/Expensify/Expensify/issues/413886
Feature Description
High Level Section: https://docs.google.com/document/d/1oLr14YhL6Y0N5g4tbozdIIrFbybBlsRA0H9I8Wm--w8/edit#bookmark=id.m0kga74f5qa8
Detailed Section: https://docs.google.com/document/d/1oLr14YhL6Y0N5g4tbozdIIrFbybBlsRA0H9I8Wm--w8/edit#bookmark=id.dnpn3lehspjl
Manual Test Steps
TBD
Automated Tests
TBD
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @The text was updated successfully, but these errors were encountered: