-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
fix: do not generate multiple invoices #28187
fix: do not generate multiple invoices #28187
Conversation
hrwX
commented
Nov 1, 2021
•
edited
edited
- fixes #28179
- checks if the invoice date is between the current invoice period.
- the invoice date was always smaller than invoice period hence there were multiple invoices being created.
Codecov Report
@@ Coverage Diff @@
## develop #28187 +/- ##
===========================================
- Coverage 55.41% 55.37% -0.05%
===========================================
Files 1248 1248
Lines 67755 67756 +1
===========================================
- Hits 37549 37517 -32
- Misses 30206 30239 +33
|
@Mergifyio backport version-13-hotfix |
🟠 Waiting for conditions to match
|
@Mergifyio backport version-13-pre-release |
🟠 Waiting for conditions to match
|
572f331
to
890a786
Compare
…cription_active_fixes
Head branch was pushed to by a user without write access
@nextchamp-saqib Can you check again, there was a failing test |
✅ Backports have been created
|
✅ Backports have been created
|
* fix: reorder updation of end date (cherry picked from commit 0f03b19) * fix: remove newline (cherry picked from commit fc375c5) * test: use `test_dependencies` instead of duplication (cherry picked from commit 656015d) * fix: create past invoices (cherry picked from commit ae657c7) * Update subscription.py (cherry picked from commit 44306bd) * fix: do not generate multiple invoices (backport frappe#28187) * chore: remove __future__ py2 compatibility imports * fix: Multicurrency invoices using subscription (cherry picked from commit 0e4842b) * fix: Add tests for multicurrency subscription (cherry picked from commit 6ab7530) * fix Co-authored-by: hrwx <himanshuwarekar@yahoo.com> Co-authored-by: Sagar Vora <sagar@resilient.tech> Co-authored-by: Deepesh Garg <deepeshgarg6@gmail.com>
* fix: reorder updation of end date (cherry picked from commit 0f03b19) * fix: remove newline (cherry picked from commit fc375c5) * test: use `test_dependencies` instead of duplication (cherry picked from commit 656015d) * fix: create past invoices (cherry picked from commit ae657c7) * Update subscription.py (cherry picked from commit 44306bd) * fix: do not generate multiple invoices (backport frappe#28187) * chore: remove __future__ py2 compatibility imports * fix: Multicurrency invoices using subscription (cherry picked from commit 0e4842b) * fix: Add tests for multicurrency subscription (cherry picked from commit 6ab7530) * fix Co-authored-by: hrwx <himanshuwarekar@yahoo.com> Co-authored-by: Sagar Vora <sagar@resilient.tech> Co-authored-by: Deepesh Garg <deepeshgarg6@gmail.com>
* fix: reorder updation of end date (cherry picked from commit 0f03b19) * fix: remove newline (cherry picked from commit fc375c5) * test: use `test_dependencies` instead of duplication (cherry picked from commit 656015d) * fix: create past invoices (cherry picked from commit ae657c7) * Update subscription.py (cherry picked from commit 44306bd) * fix: do not generate multiple invoices (backport frappe#28187) * chore: remove __future__ py2 compatibility imports * fix: Multicurrency invoices using subscription (cherry picked from commit 0e4842b) * fix: Add tests for multicurrency subscription (cherry picked from commit 6ab7530) * fix Co-authored-by: hrwx <himanshuwarekar@yahoo.com> Co-authored-by: Sagar Vora <sagar@resilient.tech> Co-authored-by: Deepesh Garg <deepeshgarg6@gmail.com>
* fix: reorder updation of end date (cherry picked from commit 0f03b19) * fix: remove newline (cherry picked from commit fc375c5) * test: use `test_dependencies` instead of duplication (cherry picked from commit 656015d) * fix: create past invoices (cherry picked from commit ae657c7) * Update subscription.py (cherry picked from commit 44306bd) * fix: do not generate multiple invoices (backport frappe#28187) * chore: remove __future__ py2 compatibility imports * fix: Multicurrency invoices using subscription (cherry picked from commit 0e4842b) * fix: Add tests for multicurrency subscription (cherry picked from commit 6ab7530) * fix Co-authored-by: hrwx <himanshuwarekar@yahoo.com> Co-authored-by: Sagar Vora <sagar@resilient.tech> Co-authored-by: Deepesh Garg <deepeshgarg6@gmail.com>