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

Gel / Fermeture : utiliser la méthode POST pour les formulaires #641

Merged
merged 7 commits into from
Nov 23, 2022

Conversation

raphodn
Copy link
Member

@raphodn raphodn commented Nov 23, 2022

Suite de #640

Quoi ?

  • créer de vrais formulaires + utiliser la méthode POST pour les actions de gel / dégel / fermeture / ré-ouverture
  • renommer l'url GET /member/show/<id> en /member/<id>

Pourquoi ?

Pourquoi POST au lieu de GET ?

  • on modifie la base de données
  • ca permettra de pouvoir passer un input dans le formulaire (e.g. pour savoir si l'on souhaite automatiquement supprimer les créneaux du membre - prochaine PR)

@raphodn raphodn self-assigned this Nov 23, 2022
@raphodn raphodn force-pushed the raphodn/member-action-forms-post branch from ffa1f7e to 2fd8fd4 Compare November 23, 2022 00:47
@raphodn raphodn force-pushed the raphodn/member-free-close-info branch from ec4f0af to e9aa7df Compare November 23, 2022 17:00
Base automatically changed from raphodn/member-free-close-info to master November 23, 2022 17:01
@raphodn raphodn force-pushed the raphodn/member-action-forms-post branch from 2fd8fd4 to 7bedfe5 Compare November 23, 2022 17:03
@raphodn raphodn merged commit 4792b7c into master Nov 23, 2022
@raphodn raphodn deleted the raphodn/member-action-forms-post branch November 23, 2022 17:10
raphodn added a commit that referenced this pull request Nov 23, 2022
raphodn added a commit that referenced this pull request Nov 25, 2022
quot17 pushed a commit to quot17/gestion-compte that referenced this pull request Nov 29, 2022
…an-grenoble#641)

* Transform member_close & member_open into POST form

* Transform member_freeze & member_unfreeze & member_freeze_change into POST form

* Remove /show in member_show url
quot17 pushed a commit to quot17/gestion-compte that referenced this pull request Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Admin : mettre à jour l'état d'un membre (gel, fermeture) avec des méthodes POST
2 participants