-
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
[HOLD for payment 2024-08-19] Updating Invoice creation flow #45173
Comments
Hi, I'm Viktoryia from Callstack - expert contributor group - and I would like to work on this issue. |
Updates: |
Updates: the PR has been opened for the review |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.18-10 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-08-19. 🎊 For reference, here are some details about the assignees on this issue:
|
Skipping the payment summary for this issue since all the assignees are employees or vendors. If this is incorrect, please manually add the payment summary SO. |
@madmax330, @VickyStash Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@madmax330, @VickyStash Eep! 4 days overdue now. Issues have feelings too... |
@madmax330, @VickyStash Now this issue is 8 days overdue. Are you sure this should be a Daily? Feel free to change it! |
@madmax330, @VickyStash 10 days overdue. Is anyone even seeing these? Hello? |
Will look into this, I think we need to pay the c+ for the PR review |
@madmax330, @VickyStash Whoops! This issue is 2 days overdue. Let's get this updated quick! |
Just waiting on c+ payment I think |
@madmax330, @VickyStash Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@madmax330, @VickyStash 6 days overdue. This is scarier than being forced to listen to Vogon poetry! |
Triggered auto assignment to @johncschuster ( |
@johncschuster for this one we just need to pay @getusha for the Pr review |
On it! |
Payment Summary: Contributor+: @getusha owed $250 via NewDot |
@madmax330, will you be adding regression test steps in this issue, or will they be added elsewhere? |
$250 approved for @getusha |
@madmax330 bump on the question above! |
We'll add them elsewhere, sorry. |
No worries! Thanks, @madmax330! I'll go ahead and close this then. |
Part of the Invoicing payments project
Main issue: https://github.com/Expensify/Expensify/issues/382977
Doc section: https://docs.google.com/document/d/1CF5gw-SJx7ckZCgTJNj1IqBnsRuSxD2Bfd_uB5rRGxk/edit#heading=h.fckbittkm504
Project: https://github.com/orgs/Expensify/projects/114
Feature Description
Relevant doc section: https://docs.google.com/document/d/1CF5gw-SJx7ckZCgTJNj1IqBnsRuSxD2Bfd_uB5rRGxk/edit#heading=h.jxlg73mn39np
Manual Test Steps
Create an invoice and make sure we're using the new flow mentioned in the design doc
Automated Tests
The text was updated successfully, but these errors were encountered: