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
feat: Add required fields for merge of invoices & order models #6196
Conversation
Added migration
Kept the Event Invoice model as it is to prevent any import breaks. Will refactor it as soon as the payments for event invoices are functional. |
@iamareebjamal @uds5501 @shreyanshdwivedi Will continue with the additional refactors as soon as this gets reviewed & merged. |
Codecov Report
@@ Coverage Diff @@
## development #6196 +/- ##
===============================================
+ Coverage 65.97% 66.01% +0.03%
===============================================
Files 288 288
Lines 14526 14532 +6
===============================================
+ Hits 9584 9593 +9
+ Misses 4942 4939 -3
Continue to review full report at Codecov.
|
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.
Looks goos
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
Please follow semantic PR conventions so that bot can assign a label and hence PR goes to appropriate category in Release Drafter |
Fixes #6195
Short description of what this resolves:
Checklist
development
branch.