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

クローズ
Darks8ヶ月前に作成 · 7件のコメント
Darks8ヶ月前 にコメント

Ç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.)
Lephenixnoir8ヶ月前 にコメント
オーナー

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.
Darks8ヶ月前 にコメント
オーナー

À 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)
Lephenixnoir8ヶ月前 にコメント
オーナー

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).
Darks6ヶ月前 にコメント
オーナー

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.
Lephenixnoir6ヶ月前 にコメント
オーナー

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:
Darks6ヶ月前 にコメント
オーナー

Voilà :smile:

Voilà :smile:
Lephenixnoir6ヶ月前 にコメント
オーナー

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...)
Lephenixnoir がクローズ 6ヶ月前
サインインしてこの会話に参加。
マイルストーンなし
担当者なし
2 人の参加者
期日

期日は未設定です。

依存関係

この課題に依存関係はありません。

読み込み中…
キャンセル
保存
まだ内容がありません