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

Bug with: cogs.croissants #13

Closed
PaulMarisOUMary opened this issue Mar 2, 2022 · 0 comments · Fixed by #18
Closed

Bug with: cogs.croissants #13

PaulMarisOUMary opened this issue Mar 2, 2022 · 0 comments · Fixed by #18
Labels
bug Something isn't working
Projects

Comments

@PaulMarisOUMary
Copy link
Owner

PaulMarisOUMary commented Mar 2, 2022

Subject:

The color of the username displayed (in the cogs.croissants) is invalid.

Reproduction steps:

Attribute a 1st role with a color to a user, then attribute a 2nd role without any colour, the 2nd role should be above the 1st.

Expected result:

The username should has the same color as the 1st role.

Actual result:

The username is black.

Aditional content:

Cause(s):

The top role is not always colored.
https://github.com/PaulMarisOUMary/Algosup-Discord/blob/b1c4087fb2821fa76b401d0c9e56b704d9778857/cogs/croissants.py#L68

How to fix:

name_color = author.color
@PaulMarisOUMary PaulMarisOUMary created this issue from a note in Algobot (To do) Mar 2, 2022
@PaulMarisOUMary PaulMarisOUMary added the bug Something isn't working label Mar 2, 2022
PaulMarisOUMary added a commit that referenced this issue Mar 7, 2022
Co-Authored-By: Léo CHARTIER <91249751+leo-chartier@users.noreply.github.com>
@PaulMarisOUMary PaulMarisOUMary mentioned this issue Mar 7, 2022
@PaulMarisOUMary PaulMarisOUMary linked a pull request Mar 7, 2022 that will close this issue
@PaulMarisOUMary PaulMarisOUMary moved this from To do to Done in Algobot Mar 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

1 participant