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

Pilotage : Publier le TB 440/485 en Occitanie et en Bretagne pour les SIAE, les DDETS, les DREETS, les CD, et la DGEFP #4931

Merged

Conversation

rsebille
Copy link
Contributor

@rsebille rsebille commented Oct 15, 2024

🤔 Pourquoi ?

https://www.notion.so/gip-inclusion/TB-440-TB-ETP-SIAE-Publier-le-TB-en-Occitanie-et-en-Bretagne-pour-les-SIAE-les-DDETS-les-DREETS-58c28a4ede4645c9ab52970cbf5b52e3

RàF :

  • Ouvrir pour les SIAE
  • Tester pour les CD/DDETS/DREETS
  • Attendre la confirmation des data et du métier

@rsebille rsebille self-assigned this Oct 15, 2024
@rsebille rsebille force-pushed the rsebille/c2/tb440-open-for-bretagne-and-occitanie branch 2 times, most recently from 84c3b94 to 43c479c Compare October 16, 2024 07:38
@rsebille rsebille changed the title WIP - Pilotage : Publier le TB 440 en Occitanie et en Bretagne pour les SIAE, les DDETS, les DREETS, et les CD Pilotage : Publier le TB 440 en Occitanie et en Bretagne pour les SIAE, les DDETS, les DREETS, et les CD Oct 16, 2024
@rsebille rsebille marked this pull request as ready for review October 16, 2024 07:53
@rsebille rsebille changed the title Pilotage : Publier le TB 440 en Occitanie et en Bretagne pour les SIAE, les DDETS, les DREETS, et les CD Pilotage : Publier le TB 440/485 en Occitanie et en Bretagne pour les SIAE, les DDETS, les DREETS, et les CD Oct 21, 2024
@rsebille rsebille changed the title Pilotage : Publier le TB 440/485 en Occitanie et en Bretagne pour les SIAE, les DDETS, les DREETS, et les CD Pilotage : Publier le TB 440/485 en Occitanie et en Bretagne pour les SIAE, les DDETS, les DREETS, les CD, et la DGEFP Oct 21, 2024
@rsebille rsebille force-pushed the rsebille/c2/tb440-open-for-bretagne-and-occitanie branch from 43c479c to 0f27df1 Compare October 21, 2024 14:52
Copy link
Contributor

@dejafait dejafait left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -176,6 +187,10 @@ def can_view_stats_dreets_iae(request):
)


def can_view_stats_dreets_iae_orga_etp(request):
return can_view_stats_dreets_iae(request) and request.current_organization.region in ["Bretagne", "Occitanie"]
Copy link
Contributor

Choose a reason for hiding this comment

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

Cette liste blanche de 2 régions est écrite en dur à quatre endroits. Je suppose que tu n'as pas utilisé une variable commune car tu envisages peut-être des évolutions indépendantes ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Non, totalement du copié/coller :).
Et je pense que le déploiement va se faire dans les mêmes départements en même temps donc je vais mutualiser !

@rsebille rsebille force-pushed the rsebille/c2/tb440-open-for-bretagne-and-occitanie branch from 0f27df1 to a491e57 Compare October 22, 2024 12:16
Copy link
Contributor

@dejafait dejafait left a comment

Choose a reason for hiding this comment

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

LGTM

@YannickPassa
Copy link
Contributor

C ti par comme disent les jeunes

@YannickPassa YannickPassa added this pull request to the merge queue Oct 25, 2024
Merged via the queue into master with commit d2767c5 Oct 25, 2024
9 checks passed
@YannickPassa YannickPassa deleted the rsebille/c2/tb440-open-for-bretagne-and-occitanie branch October 25, 2024 12:52
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.

3 participants