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

Suppression des mandats révoqués et expirés sur l'espace aidant au bout d'un an #567

Merged
merged 1 commit into from
Mar 4, 2022

Conversation

liviaribeiro
Copy link
Contributor

🌮 Objectif

Suppression des mandats révoqués et expirés sur l'espace aidant au bout d'un an.

🔍 Implémentation

Actualisation de la fonction inactive dans models pour ne pas retourner les mandats expirés ou révoqués depuis un an. Également pour la fonction _get_usagers_dict_from_mandats dans usagers.py
Création de RevokedOverYearMandatFactory et ExpiredOverYearMandatFactory pour les tests.

🏕 Amélioration continue

  • (optionnel) Une liste d'autres modifications pas en lien direct avec la PR

⚠️ Informations supplémentaires

(optionnel) Documentation, commandes à lancer, variables d'environment, etc

🖼️ Images

(optionnel) Une ou plusieurs captures d'écran

aidants_connect_web/models.py Outdated Show resolved Hide resolved
aidants_connect_web/models.py Show resolved Hide resolved
Copy link
Collaborator

@mrjmad mrjmad left a comment

Choose a reason for hiding this comment

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

je met un approval parce que si tu veux pouvoir merger les choses. Mais je pense que ca pourrait etre vraiment pas mal de mettre en place les petites simplfications proposés par @tut-tuuut .

aidants_connect_web/models.py Show resolved Hide resolved
aidants_connect_web/models.py Outdated Show resolved Hide resolved
aidants_connect_web/views/usagers.py Outdated Show resolved Hide resolved
aidants_connect_web/views/usagers.py Outdated Show resolved Hide resolved
@liviaribeiro liviaribeiro force-pushed the lpr/not_show_over_year_revoked_expired_mandates branch from 42c785c to dece799 Compare March 4, 2022 19:36
@liviaribeiro liviaribeiro force-pushed the lpr/not_show_over_year_revoked_expired_mandates branch from dece799 to 82dea5e Compare March 4, 2022 19:43
@liviaribeiro liviaribeiro merged commit 98b8213 into main Mar 4, 2022
@liviaribeiro liviaribeiro deleted the lpr/not_show_over_year_revoked_expired_mandates branch March 4, 2022 19:50
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