search

mozilla / bedrock

Making mozilla.org awesome, one pebble at a time
https://www.mozilla.org
Mozilla Public License 2.0
1.18k stars 919 forks source link

Remove use of etags and lastmod data from sitemaps #15225

Open pmac opened 1 month ago

pmac commented 1 month ago

These commits will need to be deployed one at a time. The 2nd one contains the migration to remove the data from the DB, while the first removes the use of the lastmod field.

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Project coverage is 77.96%. Comparing base (bb83e23) to head (1575c72).

Files with missing lines Patch % Lines
bedrock/sitemaps/models.py 0.00% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #15225 +/- ## ========================================== + Coverage 77.88% 77.96% +0.08% ========================================== Files 163 163 Lines 8480 8470 -10 ========================================== - Hits 6605 6604 -1 + Misses 1875 1866 -9 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

pmac commented 1 month ago

My plan was just to land this, and then push the first commit to prod, and then the 2nd.