Skip to content

fix(static): add missing bootstrap bundle sourcemap and regression test#677

Open
benmcnelly wants to merge 1 commit intofarridav:mainfrom
benmcnelly:fix-missing-bootstrap-map
Open

fix(static): add missing bootstrap bundle sourcemap and regression test#677
benmcnelly wants to merge 1 commit intofarridav:mainfrom
benmcnelly:fix-missing-bootstrap-map

Conversation

@benmcnelly
Copy link
Copy Markdown

Summary

Fix issue #674 by including the missing bootstrap.bundle.min.js.map file referenced by jazzmin/static/vendor/bootstrap/js/bootstrap.bundle.min.js.

Why

collectstatic can fail when using ManifestStaticFilesStorage or WhiteNoise's CompressedManifestStaticFilesStorage because bootstrap.bundle.min.js includes:

//# sourceMappingURL=bootstrap.bundle.min.js.map

but the referenced sourcemap was not shipped.

Changes

  • add jazzmin/static/vendor/bootstrap/js/bootstrap.bundle.min.js.map
  • add a regression test to verify the Bootstrap bundle sourcemap exists when referenced
  • align an older synthetic collectstatic test with current Django behavior

Verification

  • uv run pytest tests/test_collectstatic.py

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant