Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename reaction collector #2428

Open
BastLast opened this issue Feb 28, 2024 · 0 comments
Open

Rename reaction collector #2428

BastLast opened this issue Feb 28, 2024 · 0 comments
Labels
refactoring No visible changes but code improved
Milestone

Comments

@BastLast
Copy link
Member

BastLast commented Feb 28, 2024

Problématique de base: pourquoi ya encore une histoire de reaction collector alors que c'est des boutons ?

Les reactions collector sont des classes DraftBot maintenant, plus discord.
En gros :
On fait une commande X
Elle renvoie un collector avec :
Un ID
Des données (ReactionCollectorData)
Des réactions (ReactionCollectorReaction[])
Côté client, on affiche le collector comme il faut avec les données et réactions
Quand le joueur fait un choix (une réaction), on envoie l'index de la réaction dans l'array + l'ID, et côté Core on map ça sur le collector précédemment créé et on fait des actions
C'est vrai que le nom est pas super fou, ça aurait pu s'appeler ChoiceCollector ou je sais pas

La décision de nommage finale est donc ChoiceCollectorData et ChoiceCollectorElement

@BastLast BastLast added the refactoring No visible changes but code improved label Feb 28, 2024
@BastLast BastLast added this to the DraftBot V5 milestone Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring No visible changes but code improved
Projects
None yet
Development

No branches or pull requests

1 participant