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

Supprime la saisie de dates globales sur l'arrêté #1011

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

florimondmanca
Copy link
Collaborator

@florimondmanca florimondmanca commented Oct 17, 2024

Cette PR retire les champs "Date de début" et "Date de fin" dans le formulaire "Informations générales"

  • L'information "permanent / temporaire" est maintenant branchée sur la catégorie, et non plus la date de fin globale
  • Les dates de début et de fin globales sont calculées pour l'affichage à partir des dates des périodes
  • Une mesure doit maintenant obligatoirement avoir au moins une période
  • Toutes les intégrations de données ont été retestées en local
  • Rebase pour prendre en compte Déplace les périodes d'application dans une section à part #1029

Ça a nécessité de corriger les fixtures car il manquait des périodes sur un certain nombre d'arrêtés

@florimondmanca florimondmanca force-pushed the feat/no-regulation-dates branch 2 times, most recently from e7fb12f to 8d1343e Compare October 21, 2024 09:11
@florimondmanca florimondmanca force-pushed the feat/no-regulation-dates branch 12 times, most recently from 041d60f to 687cbd2 Compare October 23, 2024 13:01
@codecov-commenter
Copy link

codecov-commenter commented Oct 23, 2024

Codecov Report

Attention: Patch coverage is 99.05660% with 1 line in your changes missing coverage. Please review.

Project coverage is 99.00%. Comparing base (6d39082) to head (c3898ef).

Files with missing lines Patch % Lines
...oller/Regulation/DuplicateRegulationController.php 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1011      +/-   ##
============================================
- Coverage     99.00%   99.00%   -0.01%     
+ Complexity     1672     1671       -1     
============================================
  Files           329      329              
  Lines          7238     7224      -14     
============================================
- Hits           7166     7152      -14     
  Misses           72       72              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@florimondmanca florimondmanca force-pushed the feat/no-regulation-dates branch 2 times, most recently from f8e9012 to beaaf28 Compare October 23, 2024 14:12
@florimondmanca florimondmanca marked this pull request as ready for review October 23, 2024 14:13
@florimondmanca
Copy link
Collaborator Author

@mmarchois @Lealefoulon C'est prêt pour une première review

À dispo si besoin d'un topo 🙏

@mmarchois
Copy link
Collaborator

Je regarderai ça demain avec un cerveau un peu plus disponible...

@florimondmanca
Copy link
Collaborator Author

Oui je compatis @mmarchois 😅 Merci

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Supprimer la saisie de date d'arrêté
3 participants