Closed mehaase closed 3 months ago
Attention: Patch coverage is 0%
with 19 lines
in your changes are missing coverage. Please review.
Project coverage is 22.90%. Comparing base (
78f0237
) to head (8b64da2
).
Files | Patch % | Lines |
---|---|---|
src/mappings_explorer/template.py | 0.00% | 11 Missing :warning: |
src/mappings_explorer/site_builder.py | 0.00% | 8 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This PR has been published to https://mappingsexplorer.z13.web.core.windows.net/MAPEX-190-optimize-large-files/
@tiffb can you take a quick look at this? Here's an example, this page was 3MB before and is now 300KB. https://mappingsexplorer.z13.web.core.windows.net/MAPEX-190-optimize-large-files/external/nist/
@mehaase "All Mappings" table is not appearing for Azure (https://mappingsexplorer.z13.web.core.windows.net/MAPEX-190-optimize-large-files/external/azure/):
Azure "All Mappings" table is not loading.
Can you try refreshing? I am seeing it when I click that link.
If a mappings table has over 500 rows, we now truncate to 500 rows, show a message, and link to the full size file. This improves initial page load, expecially for NIST page which is the first on featured on the home page and is our most popular project.