Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove Contentful app, templates and related CSS bundles #15843

Open
5 tasks
alexgibson opened this issue Jan 10, 2025 · 1 comment · May be fixed by #15845
Open
5 tasks

Remove Contentful app, templates and related CSS bundles #15843

alexgibson opened this issue Jan 10, 2025 · 1 comment · May be fixed by #15845
Assignees
@stevejalim @alexgibson
Labels
Papercuts 💸 Tech debt payoff

Comments

@alexgibson
Copy link
Member

alexgibson commented Jan 10, 2025 •
edited by stevejalim
Loading

Description

I don't believe there's anything using the Contentul HTML templates in https://github.com/mozilla/bedrock/tree/main/bedrock/contentful/templates/includes/contentful

Removing these also means we can remove quite a large number of unused CSS bundles:

https://github.com/mozilla/bedrock/blob/main/media/static-bundles.json#L955-L1044

Success Criteria

  • Unused Contentful templates and bundles are removed from the codebase
  • Contentful Python code removed
  • Redirects updated (which were in the Python code for Contentful-managed pages)
  • All unit tests passing
  • Integration tests passing
@alexgibson alexgibson added the Papercuts 💸 Tech debt payoff label Jan 10, 2025
@alexgibson
Copy link
Member Author

Looks like these templates are used by ContentfulPreviewView - will check with Steve if we are safe to remove this yet.

@alexgibson alexgibson changed the title Remove Contentful templates and relates CSS bundles Remove Contentful app, templates and relates CSS bundles Jan 10, 2025
@stevejalim stevejalim changed the title Remove Contentful app, templates and relates CSS bundles Remove Contentful app, templates and related CSS bundles Jan 10, 2025
@stevejalim stevejalim linked a pull request Jan 10, 2025 that will close this issue
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stevejalim stevejalim

@alexgibson alexgibson

Labels
Papercuts 💸 Tech debt payoff
Projects
Bedrock Technical Roadmap (2025)
Status: In Progress
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove all Contentful-related code from the project mozilla/bedrock
2 participants
@stevejalim @alexgibson