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

Diviser la vue CounterClick #537

Closed
imperosol opened this issue Dec 19, 2022 · 1 comment
Closed

Diviser la vue CounterClick #537

imperosol opened this issue Dec 19, 2022 · 1 comment

Comments

@imperosol
Copy link
Contributor

imperosol commented Dec 19, 2022

La vue counter.views.CounterClick est un monstre tentaculaire de 500 lignes qui gère tout seul beaucoup trop de comportements différents. Lesdits comportements sont conditionnés par des variables dans les données POST de la requête qui se mélangent dans un ordre très peu compréhensible quand on lit pour la première fois (et les fois suivantes aussi, en fait).

Il faudrait donc la diviser entre plusieurs vues qui seraient chacune associées à une url précise :

  • / : renvoie une réponse text/html avec la page des clics
  • /product/add/ : ajoute un produit au panier du client
  • /studentcard/add/ : enregistre la carte étu du client
  • /product/remove/ : retire un produit du panier du client
  • /studentcard/remove/ : efface la carte étu du client
  • /studentcard/refill/ : recharge le compte du client
  • /cancel/ : efface le panier et redirige vers la page des barmen
  • /submit/ : valide le panier, fait payer le client, et redirige vers la page des barmen

A noter que si l'on retravaille les clics pour que le front accomplisse l'essentiel des actions jusqu'à la soumission finale, on peut retirer les vues pour ajouter et retirer des produits.

@imperosol imperosol converted this from a draft issue Dec 19, 2022
@imperosol imperosol moved this to Todo in Sith v3 Dec 20, 2022
@imperosol
Copy link
Contributor Author

En fin de compte, on n'a pas fait exactement comme ça. Mais on a dépecé cette horreur, c'est tout ce qui compte.

Cf. #959

@github-project-automation github-project-automation bot moved this from Todo to Déployé en production in Sith v3 Dec 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Déployé en production
Development

No branches or pull requests

1 participant