-
Notifications
You must be signed in to change notification settings - Fork 1
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
Improved performance of harvest list page #2083
Conversation
Size Change: -1 B (0%) Total Size: 3.45 MB
ℹ️ View Unchanged
|
Executive summary: little to no improvement in DOM rendering & print times, however 73x less data being transferred (on staging) project "aaaaaaa" which will result in faster loading times and a better user experience Current With performance improvements: Current data transferred (in ngx-datatable requests): 458.3 kB Test conditions: I used the development |
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.
good job
Fixes: #2045 Removed focused test Added comment describing why test can fail Typo fixes
b60dd4f
to
271c553
Compare
Improve performance of harvest list page
The harvest list page sends two requests and requests a lot of redundant information
Changes
pagination.directive.ts
now only makes one request (this fixes double requests on any page that uses this directive)Problems
Na
Issues
Fixes: #2045
Visual Changes
Na
TODO
Measure the performance increase (expecting >200%)
Final Checklist
npm run lint
)npm run test:all
)