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

Ensure Docker builds include any CSS needed when the CMS Admin mode is active, too #14756

Conversation

stevejalim
Copy link
Collaborator

@stevejalim stevejalim commented Jun 28, 2024


To test this, I did make build prod to build a new release image, locally, then shelled in:

$ pwd
/app
$ ls static/admin/css
autocomplete.4a81fc4242d0.css  dark_mode.ef27a31af300.css  nav_sidebar.269a1bd44627.css     rtl.css
autocomplete.css               dashboard.css               nav_sidebar.css                  vendor
base.523eb49842a7.css          dashboard.e90f2068217b.css  responsive.css                   widgets.css
base.css                       forms.c14e1cb06392.css      responsive.f6533dab034d.css      widgets.ee33ab26c7c2.css
changelists.9237a1ac391b.css   forms.css                   responsive_rtl.7d1130848605.css
changelists.css                login.586129c60a93.css      responsive_rtl.css
dark_mode.css                  login.css                   rtl.512d4b53fc59.css

I also compared this with a release image made yesterday, before this change:

$ pwd
/app
$ ls static/admin/
ls: cannot access 'static/admin/': No such file or directory

Issue / Bugzilla link

Resolves #14752 - Django Admin and RQ list fail up in CMS mode on Dev deployment

@stevejalim stevejalim added the Backend Server stuff yo label Jun 28, 2024
Copy link
Member

@robhudson robhudson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r+wc, maybe a short comment as to why that's needed for future devs wondering?

…s active, too

To test this, I did `make build prod` to build a new release image, locally,
then shelled in:

$ pwd
/app
$ ls static/admin/css
autocomplete.4a81fc4242d0.css  dark_mode.ef27a31af300.css  nav_sidebar.269a1bd44627.css     rtl.css
autocomplete.css               dashboard.css               nav_sidebar.css                  vendor
base.523eb49842a7.css          dashboard.e90f2068217b.css  responsive.css                   widgets.css
base.css                       forms.c14e1cb06392.css      responsive.f6533dab034d.css      widgets.ee33ab26c7c2.css
changelists.9237a1ac391b.css   forms.css                   responsive_rtl.7d1130848605.css
changelists.css                login.586129c60a93.css      responsive_rtl.css
dark_mode.css                  login.css                   rtl.512d4b53fc59.css

I also compared this with a release image made yesterday, before this change:

$ pwd
/app
$ ls static/admin/
ls: cannot access 'static/admin/': No such file or directory
@stevejalim stevejalim force-pushed the 14752-ensure-admin-statics-get-collected-during-docker-build branch from 5a93640 to 9918505 Compare July 1, 2024 12:00
@stevejalim stevejalim merged commit 06d24e3 into main Jul 1, 2024
5 checks passed
@stevejalim stevejalim deleted the 14752-ensure-admin-statics-get-collected-during-docker-build branch July 1, 2024 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backend Server stuff yo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Django-admin and rq-admin route fail on CMS deployment
3 participants