Ajouter les règles d'update automatique des trophées #17
Labels
No Label
Core
bug
duplicate
easy
enhancement
help wanted
invalid
performance
proposal
question
security
warning
wontfix
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: devs/PCv5#17
Loading…
Reference in New Issue
No description provided.
Delete Branch "%!s(<nil>)"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Ça consiste à ajouter les règles dans la classe
Member
, et appeler la méthode là où y'a besoin.Par contre ça se fait après que les systèmes sous-jacents soient créés (posts, programmes, tutos, etc.)
Tu aurais pu mettre les contextes comme un
*args
histoire qu'on en mette plusieurs dans le même appel quand c'est pertinent.À savoir ? Je vois pas trop de moments où tu postes plusieurs choses en même temps x)
Par exemple un trophée nombre de tests et un nombre de points. Quand tu postes un test tu peux valider les deux mais c'est des contextes différents (tu testes le nombre de points beaucoup plus souvent).
Justement, je préfère tout mettre ici, comme ça tu n'as pas à te soucier de savoir si l'update doit mettre à jour les points, les trophées, ou autre chose. D'un point de vue debug je trouve que c'est beaucoup plus clair ainsi
Cf la doc :
Sinon faut faire des contextes de type
points
,trophy-machin
, etc., mais ça devient le bordel à gérer puisqu'à chaque fois faudra faire particulièrement attention au contexte et à ne pas oublier un truc dans la liste.[Edit]
En fait avec cette méthode il suffit d'énumérer tous les types d'action qui peuvent interférer sur les trophées pour avoir une liste exhaustive de contextes, sans peur de se planter quand on implémentera les actions en question.
Ah oui, tu veux lister les actions et non les "causes d'obtention de trophées". Pas stupide du tout, et ça centralise la relation entre les premiers et les seconds dans un seul fichier. 😃
Voilà 😄
Fait sur
trophies
: 61a8eec. Il faudra affiner un peu la liste des contextes au fur et à mesure mais j'ai pense que j'ai répondu à la question ici.(Aussi il faudra remplacer les
TODO
par les vraies requêtes quand les modèles seront en place...)