Skip to content

Admin : Empêcher la validation d’une habilitation avec le type autre [GEN-2114]#4818

Merged
calummackervoy merged 2 commits intomasterfrom
calum/empecher-validation-organization-other
Sep 30, 2024
Merged

Admin : Empêcher la validation d’une habilitation avec le type autre [GEN-2114]#4818
calummackervoy merged 2 commits intomasterfrom
calum/empecher-validation-organization-other

Conversation

@calummackervoy
Copy link
Contributor

🤔 Pourquoi ?

Une organisation doit être typé plus précisement avant d'être habilitée

🍰 Comment ?

  • Ajouter des restrictions sur le bouton "habiliter"
  • Ajouter des restrictions sur la modification du kind depuis le site admin
  • Préciser les types des organisations habilitées dans la fixture prescribers.json pour éviter que les données invalides sont créés avec le deploiement en demo

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

http://localhost:8000/admin/prescribers/prescriberorganization/?kind__exact=Autre

💻 Captures d'écran

Screenshot 2024-09-23 at 16 56 31 Screenshot 2024-09-25 at 12 52 32

@calummackervoy calummackervoy added the modifié Modifié dans le changelog. label Sep 25, 2024
@calummackervoy calummackervoy self-assigned this Sep 25, 2024
@notion-workspace
Copy link

@calummackervoy calummackervoy force-pushed the calum/empecher-validation-organization-other branch from 8fb0bc0 to 2ef668f Compare September 25, 2024 13:39
…ther

feat(admin): prevent users setting org type to Other after habilitation
@calummackervoy calummackervoy force-pushed the calum/empecher-validation-organization-other branch from 2ef668f to 3b3b3c0 Compare September 26, 2024 09:59
@calummackervoy calummackervoy force-pushed the calum/empecher-validation-organization-other branch from 3b3b3c0 to 21502c6 Compare September 26, 2024 10:17
@calummackervoy calummackervoy force-pushed the calum/empecher-validation-organization-other branch from 21502c6 to 139cd35 Compare September 26, 2024 13:06
… Other

refactor(admin): authorization_status is a more reliable field

feat(PrescriberOrganization): CheckConstraint for restricting Other orgs validation

fix: invalidated tests

wording changes

wording change
@calummackervoy calummackervoy force-pushed the calum/empecher-validation-organization-other branch from 139cd35 to 415de96 Compare September 26, 2024 13:07
@calummackervoy calummackervoy added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Sep 26, 2024
@github-actions
Copy link

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@calummackervoy calummackervoy added this pull request to the merge queue Sep 30, 2024
Merged via the queue into master with commit 6ed7e17 Sep 30, 2024
@calummackervoy calummackervoy deleted the calum/empecher-validation-organization-other branch September 30, 2024 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants