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

Add digest archives #630

Merged
merged 13 commits into from
Mar 24, 2023
Merged

Add digest archives #630

merged 13 commits into from
Mar 24, 2023

Conversation

sgfost
Copy link
Contributor

@sgfost sgfost commented Mar 20, 2023

Adds digest archives up to Summer 2022 as static files (pdf)

Reworked DigestView to be a simpler list view of DigestArchives which are basically just metadata and a pointer to the static file.

DigestArchives are automatically created via a new management command, given the name of the file added to the static directory is formatted well-enough

resolves comses/planning#96

django/home/models.py Outdated Show resolved Hide resolved
django/home/models.py Outdated Show resolved Hide resolved
django/home/models.py Outdated Show resolved Hide resolved
sgfost added 8 commits March 23, 2023 12:17
- add DigestArchive model to index digests
- add management command update_digest_archive to create DigestArchive
  objects from static files
- remove mailchimp js loading/caching etc with simple listview
also give better error/success messages
- renamed DigestArchive -> ComsesDigest
- use utility classes instead of css in style tag
- grouped by year, make list items more readable
- removed mailchimp sign up link for now
@sgfost sgfost requested a review from alee March 23, 2023 20:17
Copy link
Member

@alee alee left a comment

Choose a reason for hiding this comment

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

LGTM, thanks Scott! Added a few minor changes.

@sgfost
Copy link
Contributor Author

sgfost commented Mar 24, 2023

Sweet, thanks for the style changes

@alee alee merged commit 98e5535 into comses:main Mar 24, 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.

2 participants