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(forum_conversation): remettre les annonces dans la liste des messages par défaut #662

Merged
merged 2 commits into from
Jun 10, 2024

Conversation

calummackervoy
Copy link
Contributor

Description

🎸 Afficher les annonces dans le fil des messages
🎸 Supprimer l'onglet annonce dans la vue d'un Forum

Type de changement

🎢 Nouvelle fonctionnalité (changement non cassant qui ajoute une fonctionnalité).
🎨 changement d'UI

Captures d'écran (optionnel)

Screenshot 2024-06-06 at 17 38 13

@@ -22,6 +22,7 @@ class TopicAdmin(BaseTopicAdmin):
inlines = [
PostInline,
]
list_filter = BaseTopicAdmin.list_filter + ("type",)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ce n'était pas dans le spec mais j'ai inclu en feature request, c'était utile pour tester

@calummackervoy calummackervoy changed the title update(forum_conversation): remettre les annonces dans la liste des messages par défaut feat(forum_conversation): remettre les annonces dans la liste des messages par défaut Jun 6, 2024
Copy link
Contributor

@vincentporte vincentporte left a comment

Choose a reason for hiding this comment

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

🎯 LGTM

self.assertIn(sticky, qs)
self.assertIn(topic, qs)
self.assertNotIn(unapproved_topic, qs)
Copy link
Contributor

Choose a reason for hiding this comment

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

bien vu!

@calummackervoy calummackervoy merged commit bb341cb into master Jun 10, 2024
8 checks passed
@calummackervoy calummackervoy deleted the 619-messages-annonces branch June 10, 2024 06:16
vincentporte pushed a commit that referenced this pull request Jun 13, 2024
🤖 I have created a release *beep* *boop*
---


##
[2.10.0](v2.9.2...v2.10.0)
(2024-06-13)


### Features

* **admin:** ajoute PostInline à TopicAdmin
([#657](#657))
([6f3049a](6f3049a))
* ajout du chemin de fer dans l'espace d'échanges et la documentation
([#649](#649))
([7cc5d14](7cc5d14))
* **forum_conversation/admin:** CertifiedPost verbose_name et traduction
([#666](#666))
([e9b0da6](e9b0da6))
* **forum_conversation/tests.py:** test explicite pour la seconde page
([#669](#669))
([5bea4f1](5bea4f1))
* **forum_conversation:** remettre les annonces dans la liste des
messages par défaut
([#662](#662))
([bb341cb](bb341cb))
* **forum_moderation:** enregistrement des messages bloquées
([#659](#659))
([7318b5d](7318b5d))
* **forum:** harmoniser la liste des topics
([#665](#665))
([6f8bf3c](6f8bf3c))
* **stats:** afficher les stats quotidiennes des Diagnostics Parcours
IAE
([#660](#660))
([5c74c56](5c74c56))
* **stats:** ajout de la vue historique des visiteurs mensuels
([#656](#656))
([804be3c](804be3c))
* **stats:** collecter le nombre de diag parcours iae réalisés
quotidiennement
([#658](#658))
([7205a2b](7205a2b))


### Bug Fixes

* **documentation:** améliorer l'affichage des bannières des fiches
pratiques
([#653](#653))
([3b539d2](3b539d2))
* **forum_conversation:** page 1 forcée dans les liens tags
([#668](#668))
([a150ab2](a150ab2))
* **forum_moderation:** motif de blocage illisible dans l'admin
([#672](#672))
([5480394](5480394))
* **quality:** formatage des balises `script` dans le gabarit
`base.html`
([#651](#651))
([0be3c8b](0be3c8b))
* **stats:** corrections mineures
([#661](#661))
([d3ae4fd](d3ae4fd))

---
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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FORUM] remettre les annonces dans la liste des messages par défaut
2 participants