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

Suljettu
3 kuukautta sitten avasi Darks · 7 kommenttia

Ç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 commented 3 kuukautta sitten
Omistaja

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 commented 3 kuukautta sitten
Omistaja

À 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 commented 3 kuukautta sitten
Omistaja

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 commented 1 kuukausi sitten
Omistaja

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 commented 1 kuukausi sitten
Omistaja

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 commented 1 kuukausi sitten
Omistaja

Voilà :smile:

Voilà :smile:
Lephenixnoir commented 1 kuukausi sitten
Omistaja

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...)
Sign in to join this conversation.
Ei merkkipaalua
No Assignees
2 osallistujaa
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
Peruuta
Tallenna
Sisältöä ei vielä ole.