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.

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

chore: clean up templates for TACC/Core-CMS#710 #209

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

wesleyboar
Copy link
Member

@wesleyboar wesleyboar commented Aug 29, 2023

Overview

Clean up templates with a https://github.com/TACC/Core-CMS that supports template overwrite instead of inheritance.

Related

Changes

  • load v3.12 beta
    cuz TACC/Core-CMS#710 is off of v3.12.0-beta.3
  • remove favicon load from assets_custom
    cuz it is now loaded in assets_core
  • remove demdata_cms/…/templates/nav_search.raw.html
    cuz it is never loaded anyway (INCLUDES_SEARCH_BAR = False)

Testing

  1. Deploy all sites.
  2. Verify favicon matches production.
  3. Verify page titles match production.

UI

Core-CMS will load the favicon in assets_core.

Core-CMS-Custom is not supposed to overwrite assets_core.

TACC/Core-CMS@4086324
@wesleyboar wesleyboar changed the title chore: remove favicon, forget template inheritance chore: embrace template overwrite, forget template inheritance Aug 29, 2023
@wesleyboar wesleyboar changed the title chore: embrace template overwrite, forget template inheritance chore: update templates for TACC/Core-CMS#710 Aug 29, 2023
@wesleyboar wesleyboar changed the title chore: update templates for TACC/Core-CMS#710 chore: clean up templates for TACC/Core-CMS#710 Aug 29, 2023
No reason to show the non-loading of a stylesheet.

This template is not loaded, thus the stylesheet is not loaded.
@wesleyboar wesleyboar mentioned this pull request Aug 29, 2023
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