-
Notifications
You must be signed in to change notification settings - Fork 0
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
Notification à l'ouverture d'un commentaire sur un bouquet #263
Notification à l'ouverture d'un commentaire sur un bouquet #263
Comments
Pas si simple parce qu'il va falloir contrôler la manière dont udata :
Pour référence, un screenshot d'une notification sur un jeu de données ci-dessous. Peut-être étendre le schéma des extras d'un bouquet avec quelque chose comme : "meta": {
"notifications": {
"model_name": "bouquet",
"external_url": "https://ecologie.data.gouv.fr/bouquets/{slug}/"
}
} Ainsi on peut utiliser ces informations si elles existent lors de la notification dans udata. Autre possibilité : utiliser les extras de la discussion pour porter le même payload. C'est une approche plus souple parce que :
Dans tous les cas, il faudra certainement prévoir une accept list pour les domaines vers lesquels peuvent pointer |
Personnellement j'aime bien l'idée, je pense que la 2e option est préférable effectivement ça permet de ne pas lier ça au sujet et plutôt à où été posté le message (si le message est posté depuis data.gouv on notifie sur data.gouv, si c'est publié sur transport on notifie sur transport, si c'est ecospheres on notifie avec des liens vers ecospheres…). Si c'est validé, ça peut être fait rapidement je pense (c'est pas un gros changement si on utilise les extras) |
La 2e solution me semble aussi plus souple.
Question de newb : L'accept-list pourrait-elle être liée aux domaines whitelistés pour l'auth ? |
Les messages sont traduits mais comme on ne traduit pas bouquet généralement pas de problème 😇. Parce que ce serait un peu relou de traduire sans avoir la chaine source dans le code de udata... Moins critique en effet mais comme la string actuelle porte le nom du modèle, autant rester là-dessus je pense. Pour l'auth j'y ai pensé mais ça m'étonnerait, je ne pense pas qu'on puisse savoir facilement d'où vient l'authentification du user courant. @ThibaudDauce je vais faire une PR là-dessus semaine pro :-) |
merci pour l'ouverture 😻🚌🚊 |
Lorsque un user commente un bouquet, l'owner du bouquet ne reçoit pas de notification.
Est de possible, à l'image d'un commentaire sur un jdd sur dgfr, que le owner d'un bouquet reçoive une notification par mail quant à la publication d'un commentaire sur un bouquet. SDans le cas où le bouquet est owned par une orga, alors tous les users de l'orga reçoivent la notif
lorsque un nouveau commentaire est fait dans la discussion, l'ensemble des personnes ayant aussi commenté recoivent une notification
The text was updated successfully, but these errors were encountered: