#1 Fusionner le compteur d'innovation dans celui d'xp

Closed
opened 1 year ago by Darks · 8 comments
Darks commented 1 year ago

Actuellement, on a deux compteurs de points par membre : xp (équivalent aux points de participations dans la version actuelle) et innovation (attribués pour la création de contenus dans la v5).

Le truc, c’est qu’ultimement le niveau qu’on veut afficher sur le profil (avec la courbe d’xp qu’on avait précisément affinée) ne prend qu’un argument, et du coup on se retrouve à faire level(xp + innovation) ou quelque chose du genre.

Comme le compteur d’xp reçoit de toute façon des points de plusieurs sources, je propose de tout fusionner en un seul compteur quitte à avoir sur la page de profil un détail du nombre de programmes/images/etc uploadés pour préserver leur mise en valeur.

(Proposé par @Lephenixnoir)

Actuellement, on a deux compteurs de points par membre : xp (équivalent aux points de participations dans la version actuelle) et innovation (attribués pour la création de contenus dans la v5). Le truc, c'est qu'ultimement le niveau qu'on veut afficher sur le profil (avec la courbe d'xp qu'on avait précisément affinée) ne prend qu'un argument, et du coup on se retrouve à faire `level(xp + innovation)` ou quelque chose du genre. Comme le compteur d'xp reçoit de toute façon des points de plusieurs sources, je propose de tout fusionner en un seul compteur quitte à avoir sur la page de profil un détail du nombre de programmes/images/etc uploadés pour préserver leur mise en valeur. (Proposé par @Lephenixnoir)
Darks added the
proposal
label 1 year ago
Darks started working 1 year ago
Darks stopped working 1 year ago
3s
Eragon commented 1 year ago
Collaborator

Ok, pour ça, mais régler le bug de la bdd et du dépassement les 1 Milliard d’xp Actuellement quand on dépasse les 32 bits on fait une 500 car c’est enregistré en bdd

Ok, pour ça, mais régler le bug de la bdd et du dépassement les 1 Milliard d'xp Actuellement quand on dépasse les 32 bits on fait une 500 car c'est enregistré en bdd
Darks commented 1 year ago
Owner

Heu, on ira pas dépasser le milliard xD

Totoyo, qui a actuellement le plus de points n’en est qu'à 16000. Même en considérant qu’on donne plus de points, disons 1000 fois plus pour avoir énormément de marge, on arrive à 16 millions. Y’a de quoi faire avant d’aller taper le milliard.

Heu, on ira pas dépasser le milliard xD Totoyo, qui a actuellement le plus de points n'en est qu'à 16000. Même en considérant qu'on donne plus de points, disons 1000 fois plus pour avoir énormément de marge, on arrive à 16 millions. Y'a de quoi faire avant d'aller taper le milliard.
Lephenixnoir commented 1 year ago
Owner

En gros l’idée était juste de capper dans le code parce que sinon un admin peut mettre un nombre énorme dans le champ d’xp sur la page de modification d’un profil et causer une erreur à la mise à jour de la base de données car le type est trop petit.

En gros l'idée était juste de capper dans le code parce que sinon un admin peut mettre un nombre énorme dans le champ d'xp sur la page de modification d'un profil et causer une erreur à la mise à jour de la base de données car le type est trop petit.
Darks commented 1 year ago
Owner

Ah oui. Bah c’est juste une condition à ne pas oublier x)

Ah oui. Bah c'est juste une condition à ne pas oublier x)
Lephenixnoir commented 1 year ago
Owner

Du coup, pour l’histoire de fusionner les compteurs ?

Du coup, pour l'histoire de fusionner les compteurs ?
Darks commented 1 year ago
Owner

Va pour la fusion.

Va pour la fusion.
Lephenixnoir commented 1 year ago
Owner

Du coup je m’en occupe, ce soir ou demain.

Du coup je m'en occupe, ce soir ou demain.
Lephenixnoir self-assigned this 1 year ago
Lephenixnoir commented 1 year ago
Owner

Fait par 21ebfb7d2a (j’ai oublié de mentionner l’issue dans le commit, my bad). Ce commit supprime le compteur d’innovation, tout ira donc dans le compteur d’XP.

Fait par 21ebfb7d2a (j'ai oublié de mentionner l'issue dans le commit, my bad). Ce commit supprime le compteur d'innovation, tout ira donc dans le compteur d'XP.
Lephenixnoir closed this issue 1 year ago
Sign in to join this conversation.
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.