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

feat(tags): inclure la gestion des tags dans le filtrage des sujets (recent, sans reponse, certifié) #615

Conversation

vincentporte
Copy link
Contributor

Description

🎸 L'URL /topics/?tags=devenir-cip permet de limiter l'affichage aux Topic associés au Tag correspondant.
🎸 Ajout de la capacité de conserver le tag selectionné lors de l'utilisation des filtres "récent", "sans réponse" et "certifié" (vue htmx)

Type de changement

🎢 Nouvelle fonctionnalité (changement non cassant qui ajoute une fonctionnalité).

Points d'attention

🦺 La même vue TopicListView est appelée en mode classic et en mode HTMX. Seuls les gabarits utilisés diffèrent.

Captures d'écran (optionnel)

Filtrage à partir d'un tag, et affichage du tag actif à la suite du compteur

image

Prise en compte du tag par le filtre (mode htmx)

image

@vincentporte vincentporte added the python Pull requests that update Python code label May 6, 2024
@vincentporte vincentporte self-assigned this May 6, 2024
@vincentporte vincentporte changed the title feat (tags) : inclure la gestion des tags dans le filtrage des sujets (recent, sans reponse, certifié) feat(tags) : inclure la gestion des tags dans le filtrage des sujets (recent, sans reponse, certifié) May 6, 2024
@vincentporte vincentporte changed the title feat(tags) : inclure la gestion des tags dans le filtrage des sujets (recent, sans reponse, certifié) feat(tags): inclure la gestion des tags dans le filtrage des sujets (recent, sans reponse, certifié) May 6, 2024
Copy link
Contributor

@francoisfreitag francoisfreitag left a comment

Choose a reason for hiding this comment

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

Je me serais attendu à pouvoir cliquer sur un tag pour filtrer par ce tag.

@vincentporte
Copy link
Contributor Author

Je me serais attendu à pouvoir cliquer sur un tag pour filtrer par ce tag.

Ce sera dans une PR à venir. Je préfère eviter une ambiguite dans l'affichage des forums et des tags. La #620 doit résoudre cette ambiguité ;)

@vincentporte vincentporte force-pushed the 585-tags-inclure-la-gestion-des-tags-dans-le-filtrage-des-sujets-recent-sans-reponse-certifié branch from 7f3f6fb to 0bde506 Compare May 14, 2024 15:12
@vincentporte vincentporte force-pushed the 585-tags-inclure-la-gestion-des-tags-dans-le-filtrage-des-sujets-recent-sans-reponse-certifié branch from 0bde506 to 59b65a1 Compare May 21, 2024 07:22
@vincentporte vincentporte force-pushed the 585-tags-inclure-la-gestion-des-tags-dans-le-filtrage-des-sujets-recent-sans-reponse-certifié branch from 59b65a1 to 8d0e04e Compare May 21, 2024 07:25
@vincentporte vincentporte merged commit 8cdb484 into master May 21, 2024
8 checks passed
@vincentporte vincentporte deleted the 585-tags-inclure-la-gestion-des-tags-dans-le-filtrage-des-sujets-recent-sans-reponse-certifié branch May 21, 2024 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
python Pull requests that update Python code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[TAGS] inclure la gestion des tags dans le filtrage des sujets (recent, sans reponse, certifié)
2 participants