-
Notifications
You must be signed in to change notification settings - Fork 42
Enhance article templates and optimize database queries for tagged pages #1170
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
792aa26
feat: Enhance article templates and optimize database queries for tag…
brylie 75fe738
fix: Ensure all tagged page queries include live and public filters
brylie d538c58
fix: Improve publication date display format in magazine issue template
brylie 4157a2a
refactor: Remove unnecessary DB-level ordering in tagged page queries
brylie 0c3e2b1
refactor: Use get_queryset() for LibraryItem, MagazineArticle, and Wf…
brylie c9c2482
fix: Update article reference in magazine article template to use ent…
brylie 29d3ea2
fix: Improve author display formatting and refactor issue link render…
brylie 9764ec5
refactor: Simplify queryset methods in LibraryItem, MagazineArticle, …
brylie f980f4e
fix: Update author display to use translation for "Authors" label in …
brylie eb76955
refactor: Remove redundant article queryset filtering in MagazineDepa…
brylie 35bdd54
refactor: Optimize queryset in MagazineArticle for better performance
brylie 23488ff
refactor: Optimize queryset in LibraryItem by using select_related fo…
brylie 51598d5
refactor: Improve HTML structure in magazine article summary template…
brylie 3c7a72a
refactor: Optimize queryset in MagazineArticle for related fetches an…
brylie f547f34
refactor: Improve HTML structure in magazine article summary template…
brylie c0e6f5d
refactor: Optimize queryset in LibraryItem by deferring streamfields …
brylie 6d7b4a6
refactor: Add authors to magazine article tests and improve template …
brylie 06c8750
refactor: Correctly reference specific issue object in magazine artic…
brylie 03c4ac4
refactor: Enhance author handling in template rendering tests and imp…
brylie ec5f79b
refactor: Update authors label and list to use dynamic IDs for improv…
brylie f588b10
refactor: Improve authors label handling in template rendering tests …
brylie File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| {% include "magazine/magazine_article_summary.html" with article=entity.specific %} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,11 @@ | ||
| {% load wagtailcore_tags %} | ||
| <article class="card bg-base-100 shadow-sm hover:shadow-md transition-shadow mb-4"> | ||
| <div class="card-body"> | ||
| <h3 class="card-title"> | ||
| <a href="{% pageurl entity %}">{{ entity.title }}</a> | ||
| </h3> | ||
| {% if entity.specific.publication_date %} | ||
| <p class="text-sm text-base-content/70">Published {{ entity.specific.publication_date|date:"F Y" }}</p> | ||
| {% endif %} | ||
| </div> | ||
| </article> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.