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

Fechado
aberto por Darks 3 meses atrás · 7 comentários

Ç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 comentou 3 meses atrás
Proprietário

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 comentou 3 meses atrás
Proprietário

À 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 comentou 3 meses atrás
Proprietário

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 comentou 1 mês atrás
Proprietário

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 comentou 1 mês atrás
Proprietário

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 comentou 1 mês atrás
Proprietário

Voilà :smile:

Voilà :smile:
Lephenixnoir comentou 1 mês atrás
Proprietário

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...)
Acesse para participar desta conversação.
Sem marco
Sem responsável
2 participante(s)
Data limite

Data limite não informada.

Dependências

Esta issue atualmente não tem dependências.

Carregando…
Cancelar
Salvar
Ainda não há conteúdo.