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): ajouter des étiquettes à un forum #746

Merged
merged 5 commits into from
Aug 26, 2024
Merged

Conversation

vincentporte
Copy link
Contributor

@vincentporte vincentporte commented Aug 7, 2024

Description

Lors de la création ou de la mise à jour d'un Forum :
🎸 Permettre de selectionner une ou plusieurs étiquettes
🎸 Permettre d'ajouter une ou plusieurs étiquettes inexistantes

Type de changement

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

Points d'attention

🦺 suite PR #744

Captures d'écran (optionnel)

Création d'un forum

image

Mise à jour d'un forum, avec plusieurs nouveaux tags séparés par des virgules

image

Résultat de la mise à jour

image

@vincentporte vincentporte changed the base branch from master to 726-forum-filtrer-les-forums-sur-les-tags August 7, 2024 15:21
@vincentporte vincentporte added the python Pull requests that update Python code label Aug 7, 2024
@vincentporte vincentporte linked an issue Aug 7, 2024 that may be closed by this pull request
Copy link
Contributor

@hellodeloo hellodeloo left a comment

Choose a reason for hiding this comment

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

LGTM

@vincentporte vincentporte force-pushed the 726-forum-filtrer-les-forums-sur-les-tags branch from fa8de27 to c57a79d Compare August 8, 2024 08:44
@vincentporte vincentporte force-pushed the 726-part-2 branch 2 times, most recently from 408e3b9 to abf6d36 Compare August 8, 2024 08:48
@vincentporte vincentporte force-pushed the 726-forum-filtrer-les-forums-sur-les-tags branch 2 times, most recently from 73f4f55 to d932b67 Compare August 12, 2024 14:43
@vincentporte vincentporte force-pushed the 726-part-2 branch 2 times, most recently from 766f188 to f75b8bd Compare August 13, 2024 09:09
@vincentporte vincentporte force-pushed the 726-forum-filtrer-les-forums-sur-les-tags branch from 361fffc to 87936b4 Compare August 14, 2024 09:07
Base automatically changed from 726-forum-filtrer-les-forums-sur-les-tags to master August 26, 2024 09:28
@vincentporte vincentporte merged commit 7f435bb into master Aug 26, 2024
8 checks passed
@vincentporte vincentporte deleted the 726-part-2 branch August 26, 2024 10:12
vincentporte added a commit that referenced this pull request Aug 26, 2024
## Description

🎸 Réduire la liste des `Forum` enfants affichés en filtrant sur leurs
tags

suite #746 

## Type de changement

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

### Points d'attention

🦺 Affichage des enfants, les petits-enfants ne sont pas considérés,
qu'il possède le tag ou non

### Captures d'écran (optionnel)


![image](https://github.com/user-attachments/assets/6042fdf1-90e6-4e20-abd9-829bab8e8d0f)
vincentporte added a commit that referenced this pull request Sep 5, 2024
gi## Description

🎸 Afficher les filtres sur les `tags` des `Forum` enfants
(`descendants`)
🎸 Mettre à jour la liste des `Forum` enfants par une vue `ListView` HTMX
🎸 Seuls les `tags` associés aux `Forum` enfant sont affichés
🎸 Le filtrage n'est possible que sur un seul `tag` à la fois.

🦺 suite des PR #746 #750 #751 

## Type de changement

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

### Points d'attention

🦺 choix d'une vue indépendante pour éviter la surcharge de `ForumView`.
Conséquence, ne permet pas la mise à jour dynamique de l'url avec le
paramètre `hx-push`
🦺 mise à jour de `parse_response_to_soup` : gestion de `hx-get` en plus
de `hx-post` et `href`
🦺 mise à jour de la command `populate` : ajout de tags aux `Forum`
générés
🦺 suppression du template `partials/tag.html`, mutualisation inutile
avec l'affichage des `tags` des `Topics`

⚠️ #750 installait la capacité de la vue à recevoir plusieurs tags en
paramètre de la requete. cette capacité est retirée ici car non
utilisable par l'UI

### Captures d'écran (optionnel)

Categorie avec fiches sans tag


![image](https://github.com/user-attachments/assets/cc96b706-0235-41e9-8755-8657543c794e)


Categorie avec fiches taggées non filtrées


![image](https://github.com/user-attachments/assets/af981989-dcbb-4f90-b76f-2a6dcaeeebc8)


Categorie avec fiches taggées filtrées


![image](https://github.com/user-attachments/assets/b8eabf50-c861-472b-a690-7cb4fff34614)
vincentporte pushed a commit that referenced this pull request Sep 9, 2024
🤖 I have created a release *beep* *boop*
---


##
[2.13.0](v2.12.0...v2.13.0)
(2024-09-05)


### Features

* **event:** amélioration de l'admin
([#753](#753))
([ca56d83](ca56d83))
* **forum:** retrait des communautés privées
([#768](#768))
([1c120e3](1c120e3))
* **partner:** ajout des pages partenaires
([#757](#757))
([33fd1c6](33fd1c6))
* **partner:** améliorations post déploiement
([#763](#763))
([09a865f](09a865f))
* **partner:** lien avec les fiches pratiques
([#761](#761))
([60bb2ee](60bb2ee))
* **partner:** referencer les fiches partenaires dans sitemap.xml
([#759](#759))
([91bea96](91bea96))
* **tags:** afficher les filtres sur les tags
([#752](#752))
([7a10488](7a10488))
* **tags:** afficher les Tags dans la liste des fiches pratiques d'une
categorie
([#751](#751))
([ed005b7](ed005b7))
* **tags:** ajouter des étiquettes à un forum
([#746](#746))
([7f435bb](7f435bb))
* **tags:** ajouter une étiquette inexistante à un sujet
([#744](#744))
([b82ea73](b82ea73))
* **tags:** filtrer les `forum` enfants à partir des tags
([#750](#750))
([eb6683a](eb6683a))


### Bug Fixes

* **forum_conversation:** Résolution des erreurs de tests sur les
contenus des messages
([#754](#754))
([8e0f35d](8e0f35d))
* **partner:** mise en forme des logos de la vue en liste
([#764](#764))
([7a84319](7a84319))
* **post:** simplification du sujet des `Post`
([#748](#748))
([3c2a5a0](3c2a5a0))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

[FORUM] filtrer les forums sur les tags
2 participants