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

Unifier ce qu'on utilise pour les pseudos en draftbot v5 #2654

Open
BastLast opened this issue Nov 14, 2024 · 4 comments
Open

Unifier ce qu'on utilise pour les pseudos en draftbot v5 #2654

BastLast opened this issue Nov 14, 2024 · 4 comments
Labels
refactoring No visible changes but code improved

Comments

@BastLast
Copy link
Member

Faut étudier et décider ce qui est le mieux parce que actuellement on utilise 50 trucs différents:

image

est-ce qu'on escape ? est-ce que c'est username ou displayname ? etc...

@BastLast BastLast added the refactoring No visible changes but code improved label Nov 14, 2024
@BastLast BastLast added this to the DraftBot V5 milestone Nov 14, 2024
@BastLast
Copy link
Member Author

image
ya aussi ça qui est encore un truc différent ??

@Feiryn
Copy link
Contributor

Feiryn commented Nov 14, 2024

Vaut mieux toujours utiliser user.attributes.gameUsername si on l'a. Sinon, cette valeur est égale à user.displayName pour un user discord (donc dans notre cas 95% du temps).
Quand il y aura des joueurs autre que discord, ce sera le nom définit lors de l'inscription donc user.attributes.gameUsername

@BastLast
Copy link
Member Author

Okay well faut bien unifier ca dans le code avant de release, ca sera une easy task a faire a la fin :)

@Feiryn
Copy link
Contributor

Feiryn commented Nov 14, 2024

Je déplace dans la milestone adaptée

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

2 participants