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

[Notifications] Correction des notifications de nouveau signalement et d'accès à la fiche pour le 69 #3523

Merged
merged 2 commits into from
Jan 9, 2025

Conversation

emilschn
Copy link
Collaborator

@emilschn emilschn commented Jan 7, 2025

Ticket

#3518

Description

Dans le 69, via leur configuration particulière, l'ensemble des RT est notifié pour les nouveaux signalements, alors que le territoire est divisé en deux partenaires.
De plus, en accédant via l'url directe, ils pouvaient voir les signalements, ce qui ne devrait pas être le cas.

Changements apportés

  • Modification du voter pour accéder aux signalements pour prendre en compte la config des codes insee autorisés
  • Modification de l'enregistrement des draft pour enregistrer la valeur du code insee depuis le front, et l'avoir à disposition pour filtrer les notifications

Pré-requis

Utiliser la base de prod pour ne pas avoir à dupliquer les configurations.
En base, modifier le mot de passe d'un utilisateur dans le partenaire COR

Tests

  • Créer un signalement sur Lyon : seuls les utilisateurs de Lyon Metropole doivent être notifiés
  • Essayer d'accéder au signalement avec l'utilisateur du partenaire COR via l'uuid : on doit être bloqué
  • Créer un signalement dans la COR (Ex : Affoux) : seuls les utilisateurs de la COR doivent être notifiés

Copy link

sonarqubecloud bot commented Jan 7, 2025

Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

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

ok

Copy link
Collaborator

@hmeneuvrier hmeneuvrier left a comment

Choose a reason for hiding this comment

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

Ok relecture et tests

@hmeneuvrier hmeneuvrier merged commit 05f9755 into main Jan 9, 2025
6 checks passed
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.

3 participants