Skip to content

Admin: Remplacer l’erreur 500 en cas de refus d'habilitation par un message plus explicite#4747

Merged
tonial merged 2 commits into
masterfrom
alaurent/refuse_hab
Sep 19, 2024
Merged

Admin: Remplacer l’erreur 500 en cas de refus d'habilitation par un message plus explicite#4747
tonial merged 2 commits into
masterfrom
alaurent/refuse_hab

Conversation

@tonial

@tonial tonial commented Sep 14, 2024

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Je me suis rendu compte que les tests étaient bizarres car les subTests ne font rien de différent (les arguments ne sont pas utilisés par le formulaire).
Je ne suis pas certain de ce qu'on veut faire à la place (vu qu je ne suis pas certain de ce que les SubTests voulaient tester)

image

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

@tonial tonial added the modifié Modifié dans le changelog. label Sep 14, 2024
@tonial tonial requested a review from xavfernandez September 14, 2024 05:18
@tonial tonial self-assigned this Sep 14, 2024
@tonial tonial changed the title Admin: Remplacer l’erreur 500 en cas de refus d'habilitation par un message plus explicite Admin: Remplacer l’erreur 500 en cas de refus d'habilitation par un message plus explicite (GEN-2035] Sep 14, 2024
@notion-workspace

Copy link
Copy Markdown

@tonial tonial changed the title Admin: Remplacer l’erreur 500 en cas de refus d'habilitation par un message plus explicite (GEN-2035] Admin: Remplacer l’erreur 500 en cas de refus d'habilitation par un message plus explicite [GEN-2035] Sep 14, 2024
@tonial tonial changed the title Admin: Remplacer l’erreur 500 en cas de refus d'habilitation par un message plus explicite [GEN-2035] Admin: Remplacer l’erreur 500 en cas de refus d'habilitation par un message plus explicite Sep 14, 2024
@notion-workspace

Copy link
Copy Markdown

@tonial tonial force-pushed the alaurent/refuse_hab branch from 8eff83e to 25b4b7e Compare September 14, 2024 05:23
Comment thread tests/prescribers/tests.py Outdated
Comment thread tests/prescribers/tests.py Outdated
Comment thread tests/prescribers/tests.py Outdated
The is_authorized and authorization_status are not used in the form (and
not transmitted in the real admin form)

I'm not sure what this rights_list was supposed to do....
Was it supposed to check whether non super admin could see the REFUSED
button ?
@tonial tonial force-pushed the alaurent/refuse_hab branch from 25b4b7e to 3340503 Compare September 18, 2024 04:19
@tonial tonial added this pull request to the merge queue Sep 19, 2024
Merged via the queue into master with commit 7ace54d Sep 19, 2024
@tonial tonial deleted the alaurent/refuse_hab branch September 19, 2024 04:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants