-
Notifications
You must be signed in to change notification settings - Fork 67
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
Zero amount invoices & debit notes #1037
Zero amount invoices & debit notes #1037
Conversation
693880b
to
91c4792
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Tested all examples and they are working, except for debit_note_flow
on dummy
.
Guess that was already broken so we can fix that in a separate PR.
Your gist also has a small error now:
curl https://gist.githubusercontent.com/pnowosie/6553be29d2cfd58c2ef5947334d719d7/raw/1276966f6cb6d7efc34b6878207a3015d353e89d/invoice_flow_example.patch | git apply -v
Arbiter::spawn(async move { | ||
std::thread::sleep(Duration::from_millis(100)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
small nit: would not import this at the top of the file or only 1 use
std::thread::sleep(Duration::from_millis(100)); | |
std::thread::sleep(actix::clock::Duration::from_millis(100)); |
std::thread::sleep(Duration::from_millis(100)); | |
std::thread::sleep(Duration::from_millis(100)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggest commenting the crucial lines in the code, otherwise alright.
PAY-182
I've modified
invoice_flow
example [LINK],debit_note_flow
modified accordingly.Result of the db query