-
Notifications
You must be signed in to change notification settings - Fork 88
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
custom management command that sets an announcement banner on front page #1781
Conversation
needs more styling
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Bomme! I left a couple of comments and I'll work on the styling part. I also think there's some missing change in the view of the front page where the contents of announcement_cache
are passed to the front page template. I guess you did not commit that file?
@ffont I addressed your comments and added the missing line! |
Cool! I added the styling. Also renamed the management command to Once you test it and tell me it is ok (and try with texts for your announcement), then we can merge, deploy and add your banner. |
needs more styling
Issue(s)
Description
Deployment steps: