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

Cleanup Cruft & Be Consistent #221

Open

wesleyboar opened this issue Sep 1, 2023 · 0 comments

Member

wesleyboar commented Sep 1, 2023 •

edited

Loading

Overview

Update all projects based on general review comments in #184.

To Do

Elasticsearch

Example: b942d2c

remove elasticsearch.py
remove elasticsearch container volume ./elasticsearch.py:…

Custom Apps

Example: a993361, f8e4640

remove cms container volume ./src/apps:/code/apps
remove Docker COPY /src/apps /code/apps
remove src/apps

TACC Core-Styles Version

Example: d192e40

add TACC_CORE_STYLES_VERSION to taccsite_cms/settings_custom.py

CMS Settings

make order of settings in settings_custom.py match Core-CMS

No Extra init.py

Example: /example_cms& /ecep-cms

remove extra __init__.py files (only apps need them)

Remove Placeholder Templates

Example: demdata-cms

Edit all pages on a production website of a deployed Core-CMS-Custom app.
Update them to not use DEPRECATED templates.
Delete the ____-cms/templates directory.

Just One Images Folder

Example: 30c248

move images from /img/org_logos & /img/favicons to just /img
update settings_custom.py to use /img path, not /img/org_logos & /img/favicons

The text was updated successfully, but these errors were encountered:

wesleyboar mentioned this issue

feat: ecep_cms #184

Merged

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