-
Notifications
You must be signed in to change notification settings - Fork 490
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: Show Ticket Sales in Events List #1315
feat: Show Ticket Sales in Events List #1315
Conversation
Codecov Report
@@ Coverage Diff @@
## development #1315 +/- ##
==============================================
Coverage ? 24.79%
Complexity ? 757
==============================================
Files ? 236
Lines ? 8538
Branches ? 342
==============================================
Hits ? 2117
Misses ? 6338
Partials ? 83
Continue to review full report at Codecov.
|
dd854c2
to
d64e0e5
Compare
@iamareebjamal Status: The Server has bugs for endpoint |
@MishuVS What's the status of resolving the bug in server ? Please make it a priority if it isn't completed yet. That is a blocker for this PR. |
7a74497
to
e2de1bf
Compare
@iamareebjamal @MishuVS @sridharjajoo Server side bug is resolved and the PR is ready for review. |
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.
It's highly doubtful that event creation will work after addition of the new fields in the events model. Can you please test it ?
@srv-twry The event creation is working fine. |
5117d46
to
76f532d
Compare
Fixes #1151
Checklist:
Changes:
Screenshots for the change: