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

Tags view refactor #1851

Open

Bomme wants to merge 2 commits into master from tags_view_refactor

Contributor

Bomme commented Feb 18, 2025

Issue(s)
#1845 fixed an issue that was not detected by our tests

Description
I started out covering /browse/tags with tags after the above mentioned issue and ended up refactoring major parts of it.

tag cloud: this is now a separate template as it had no connection to the search page.
1. removed special case handling for initial_tagcloud from search.html
2. add templates/tags/tag_cloud.html
introduced a separate view function for the multiple tag look-up --> redirect

Deployment steps:
none

Bomme added 2 commits

February 18, 2025 14:48


          Refactor tag browsing to separate tag cloud from "tags_mode" search

be995e0


          Fix deprecated timezone import

a57fa1e

Bomme requested a review from ffont

February 18, 2025 13:59

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

Labels

None yet