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

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

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

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

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

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

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

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

Voilà :smile:

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

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

期日は未設定です。

依存関係

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

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