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: Fréquence de mise à jour des services #159

Open
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

ggounot
Copy link
Contributor

@ggounot ggounot commented Dec 30, 2024

https://trello.com/c/eGapE2J2/111-service-en-besoin-de-maj-choix-sur-la-fiche-service

  • La fréquence de mise a jour est implémentée avec un modèle de type TextChoices s'appelant UpdateFrequency et d'un champ update_frequency ajouté au modèle Service ;
  • À la fois les services et les modèles bénéficient de ce champ update_frequency ;
  • La méthode ServiceManager.update_advised() a été mise à jour pour prendre en compte le champ update_frequency ;
  • Des tests existants ont été converti au format pytest et enrichis ;
  • Le texte de l'e-mail a été un peu modifié pour enlever la partie « Tous les 6 mois, ».

Création/édition d'un service :
image

Admin :
image

@ggounot ggounot force-pushed the feature/service-frequence-mise-a-jour branch from 01cf572 to 6090d37 Compare December 30, 2024 16:06
@ggounot ggounot changed the title Fréquence de mise à jour des services feat: Fréquence de mise à jour des services Dec 30, 2024
@ggounot ggounot added the feature New feature or request label Dec 30, 2024
@ggounot ggounot force-pushed the feature/service-frequence-mise-a-jour branch 2 times, most recently from 1fb3372 to b9ef55a Compare January 2, 2025 18:07
@ggounot ggounot force-pushed the feature/service-frequence-mise-a-jour branch from b9ef55a to b780650 Compare January 2, 2025 18:09
@ggounot ggounot marked this pull request as ready for review January 2, 2025 19:21
@ggounot ggounot requested review from ikarius and jbuget January 2, 2025 19:22
@ggounot ggounot self-assigned this Jan 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant