#17 Ajouter les règles d'update automatique des trophées

Fermé
créé il y a 3 mois par Darks · 7 commentaires
Darks a commenté il y a 3 mois

Ç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.)

Ç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.)
Lephenixnoir a commenté il y a 3 mois
Propriétaire

Tu aurais pu mettre les contextes comme un *args histoire qu’on en mette plusieurs dans le même appel quand c’est pertinent.

Tu aurais pu mettre les contextes comme un `*args` histoire qu'on en mette plusieurs dans le même appel quand c'est pertinent.
Darks a commenté il y a 3 mois
Propriétaire

À savoir ? Je vois pas trop de moments où tu postes plusieurs choses en même temps x)

À savoir ? Je vois pas trop de moments où tu postes plusieurs choses en même temps x)
Lephenixnoir a commenté il y a 3 mois
Propriétaire

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).

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).
Darks a commenté il y a 1 mois
Propriétaire

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 :

        Auto-update trophies for the current user. Please use one of the
        following contexts when possible:
          - new-post
          - new-program
          - new-tutorial
          - new-test
          - new-event-participation
          - new-picture
          - on-program-reward
          - on-login
          - on-profile-update

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.

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 : ``` Auto-update trophies for the current user. Please use one of the following contexts when possible: - new-post - new-program - new-tutorial - new-test - new-event-participation - new-picture - on-program-reward - on-login - on-profile-update ``` 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.
Lephenixnoir a commenté il y a 1 mois
Propriétaire

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. :smiley:

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. :smiley:
Darks a commenté il y a 1 mois
Propriétaire

Voilà :smile:

Voilà :smile:
Lephenixnoir a commenté il y a 1 mois
Propriétaire

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…)

Fait sur `trophies` : [61a8eec](https://gitea.planet-casio.com/devs/PCv5/commit/61a8eeccbae3a103bb4cb44f8aca3851b577f5d3). 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...)
Connectez-vous pour rejoindre cette conversation.
Aucun jalon
Pas d'assignataires
2 participants
Échéance

Aucune échéance n'a été définie.

Dépendances

Ce ticket n'a actuellement pas de dépendance.

Chargement…
Annuler
Enregistrer
Il n'existe pas encore de contenu.