macports / macports-webapp

Web application to display information about ports, build history and installation statistics
https://ports.macports.org/
BSD 2-Clause "Simplified" License
56 stars 27 forks source link

Port health: Use background loading #327

Closed arjunsalyan closed 2 years ago

arjunsalyan commented 2 years ago

This improves performance for the port details page because port-health is an expensive query.

codecov[bot] commented 2 years ago

Codecov Report

Merging #327 (d46d702) into main (fc36171) will decrease coverage by 0.09%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #327      +/-   ##
==========================================
- Coverage   81.45%   81.35%   -0.10%     
==========================================
  Files         111      111              
  Lines        2750     2752       +2     
  Branches      214      214              
==========================================
- Hits         2240     2239       -1     
- Misses        464      467       +3     
  Partials       46       46              
Impacted Files Coverage Δ
port/views.py 67.60% <0.00%> (-1.69%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update fc36171...d46d702. Read the comment docs.

mascguy commented 2 years ago

Arjun, thanks again for all of the work you've done - and continue to do - for the site. Great stuff!

arjunsalyan commented 2 years ago

While this has improved the loading time of port-details page, port health is still taking ages to load. That query needs to be improved.