-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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: adding table for admin-sales-by-event #5243
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/eventyay/open-event-frontend/fsv8649hw |
Codecov Report
@@ Coverage Diff @@
## development #5243 +/- ##
===============================================
- Coverage 22.75% 22.72% -0.04%
===============================================
Files 490 491 +1
Lines 5207 5215 +8
Branches 36 36
===============================================
Hits 1185 1185
- Misses 4017 4025 +8
Partials 5 5
Continue to review full report at Codecov.
|
Please group the status sales like in previous UI and use standard date cell like in invoice table |
app/templates/components/ui-table/cell/admin/sales/cell-amount.hbs
Outdated
Show resolved
Hide resolved
Date is still in wrong format. Please check all points before finalizing |
Amount column doesn't show anything, just 0 always. And build is also failing |
Please see last commit. Many changes were unnecessary |
Fixes #2794
Fixes #5163
Checklist
development
branch.