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): filtrer les questions dans l'espace d'échanges #681

Merged
merged 7 commits into from
Jun 24, 2024

Conversation

vincentporte
Copy link
Contributor

@vincentporte vincentporte commented Jun 17, 2024

Description

🎸 Rediriger les demandeurs d'emploi et les utilisateurs du site des emplois vers les services adéquats pour réduire le nombre de questions 'non désirées'

Type de changement

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

Points d'attention

🦺 l'accès à la vue TopicCreateView est direct lorsque le forum a la propriété is_in_documentation_area ou le paramètre checked=1 dans l'url.

Captures d'écran (optionnel)

image

image

"forum_pk": cls.forum.pk,
},
)
+ "?checked=1"
Copy link
Contributor

Choose a reason for hiding this comment

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

OK!

Comment on lines +44 to +53
def get(self, request, *args, **kwargs):
forum = self.get_forum()
if forum.is_in_documentation_area or self.request.GET.get("checked"):
return super().get(request, *args, **kwargs)
return redirect(
reverse(
"forum_conversation_extension:topic_create_check",
kwargs={"forum_pk": forum.pk, "forum_slug": forum.slug},
)
)
Copy link
Contributor

Choose a reason for hiding this comment

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

Efficace :)

@vincentporte vincentporte force-pushed the 679-forumconversation-poser-une-question branch from c88750b to 26f58d5 Compare June 24, 2024 15:52
@vincentporte vincentporte merged commit 48cdd07 into master Jun 24, 2024
8 checks passed
@vincentporte vincentporte deleted the 679-forumconversation-poser-une-question branch June 24, 2024 15:55
vincentporte pushed a commit that referenced this pull request Jul 4, 2024
🤖 I have created a release *beep* *boop*
---


##
[2.11.0](v2.10.0...v2.11.0)
(2024-07-03)


### Features

* **admin:** améliorer le filtrage des stats de forum
([#699](#699))
([cd0d301](cd0d301))
* **admin:** filtrage des stats de forum, ajout tri
([#701](#701))
([1da74df](1da74df))
* **forum_conversation:** filtrer les questions dans l'espace d'echanges
- stats de suivi
([#700](#700))
([bac6e55](bac6e55))
* **forum_conversation:** filtrer les questions dans l'espace d'échanges
([#681](#681))
([48cdd07](48cdd07))
* **forum:** Activation de l'AB Test Notation des forums
([#686](#686))
([c5699d4](c5699d4))
* **forum:** collecter les notations des fiches pratiques
([#677](#677))
([4c39290](4c39290))
* **home:** MVP zone editoriale sur la page d'accueil
([#703](#703))
([5e3f759](5e3f759))
* **home:** petites améliorations
([#689](#689))
([7471451](7471451))
* **notification:** nouveau système des notifications messages
([#697](#697))
([f2eedd4](f2eedd4))
* **stats:** collecter les stats d'activité hebdo des forums de la
documentation
([#691](#691))
([ab513a8](ab513a8))
* **stats:** page hebdo partenaires fiches pratiques
([#695](#695))
([f7f26b8](f7f26b8))


### Bug Fixes

* **cron.json:** erreur copier-coller sur send_notifications_daily.sh
([#705](#705))
([32f59ed](32f59ed))
* **forum:** ab test forum rating
([#687](#687))
([f0e9c2b](f0e9c2b))
* **forum:** ab test forum rating
([#688](#688))
([d3027e4](d3027e4))
* **notification:** mettre à jour le sent_at pour éviter les
notifications duplicates
([#707](#707))
([fca7f29](fca7f29))
* **stats:** derniere date collectée des stats matomo
([#684](#684))
([015e08b](015e08b))
* **stats:** gel du temps dans les tests de la vue
`StatistiquesPageView`
([#702](#702))
([df66524](df66524))

---
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>
vincentporte added a commit that referenced this pull request Aug 6, 2024
## Description

🎸 suite #681
🎸 le parametre `?checked=1` génère du bruit dans la suite de test.
simplification en `?checked`


## Type de changement

🪲 Correction de bug (changement non cassant qui corrige un problème).
🚧 technique
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 CONVERSATION] poser une question
2 participants