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(notification): passage des notifications en lues #712

Merged
merged 3 commits into from
Aug 6, 2024

Conversation

calummackervoy
Copy link
Contributor

Description

🎸 Quand un utilisateur voit un sujet, les notifications liés sont marqués lues
🎸 Adjustment des champs inclus dans la classe NotificationAdmin

Type de changement

🚧 technique

Captures d'écran (optionnel)

Screenshot 2024-07-04 at 17 45 45

@calummackervoy calummackervoy added the python Pull requests that update Python code label Jul 4, 2024
@calummackervoy calummackervoy self-assigned this Jul 4, 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.

C'est propre et efficace.
Il resterait à attraper les utilisateurs anonymes qui ont des notifs en attente ;)

lacommunaute/notification/tests/tests_models.py Outdated Show resolved Hide resolved
lacommunaute/notification/tests/tests_models.py Outdated Show resolved Hide resolved
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

@calummackervoy calummackervoy merged commit a1a6b73 into master Aug 6, 2024
8 checks passed
@calummackervoy calummackervoy deleted the 706-notifications-prevention branch August 6, 2024 08:55
vincentporte pushed a commit that referenced this pull request Aug 12, 2024
🤖 I have created a release *beep* *boop*
---


##
[2.12.0](v2.11.0...v2.12.0)
(2024-08-07)


### Features

* **documentation:** rendre la certification optionnelle
([#741](#741))
([6141112](6141112))
* **footer:** mise à jour des liens sociaux
([#730](#730))
([f19602e](f19602e))
* **forum:** attr accept définit sur forms.ImageField
([#734](#734))
([0e8b8c9](0e8b8c9))
* **forum:** fin de l'AB test sur la notation des forums
([#708](#708))
([6705ead](6705ead))
* **forum:** utiliser l'image du forum dans la vignette opengraph
([#725](#725))
([cc137c1](cc137c1))
* mise à jour entête, pied de page et menu
([#739](#739))
([4493267](4493267))
* **notification:** passage des notifications en lues
([#712](#712))
([a1a6b73](a1a6b73))
* **seo:** mise à jour de l'image OpenGraph
([#720](#720))
([f37521c](f37521c))


### Bug Fixes

* **forum:** balise `article` et dimension des images
([#737](#737))
([b1ed91f](b1ed91f))
* mise à jour de la public key sentry pour l'envoi des logs
([#732](#732))
([616c0aa](616c0aa))
* **upvotes:** largeur du bouton s'abonner dynamique
([#743](#743))
([32cb339](32cb339))
* **upvote:** supprimer un parametrage ambigue
([#742](#742))
([168df13](168df13))

---
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.

[NOTIFICATION] passer les notifications en lues qd un utilisateur répond
2 participants