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

ShowMeYourName - Fixed possible false positive bug #2633

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

Jono997
Copy link

@Jono997 Jono997 commented Jun 27, 2024

There seems to be a bug where when a user's name has capital letters, this plugin always acts as if the username and nickname are different. This is my attempt to fix it.

There seems to be a bug where when a user's name has capital letters, this plugin always acts as if the username and nickname are different. This is my attempt to fix it.
@RealityWinner
Copy link

RealityWinner commented Jun 27, 2024

@Bloofield This broke due to your previous PR #2630

@Bloofield
Copy link
Contributor

@Bloofield This broke due to your previous PR #2630

Interesting. I assumed usernames would always be lowercase which is why I didn't add the lowercase conversion for usernames

@Jono997
Copy link
Author

Jono997 commented Jun 28, 2024

So will this be merged or the other PR reverted?

@Bloofield
Copy link
Contributor

@Jono997 This one should be merged. There needs to be a lowercase conversion to prevent duplicate usernames from showing up.

The thought behind the original PR was to prevent nicknames with capitalisation in them from being shown.

@hideki2k02
Copy link

Up on this. Just waiting for this to get merged so I can make a few changes too

@RealityWinner
Copy link

@Vendicated can you please merge this and fix the broken plugin since #2630 you merged the other day

@Jono997
Copy link
Author

Jono997 commented Jul 3, 2024

@Vendicated Sorry for bugging you again, but can you please take a look at this?

maddie480 added a commit to maddie480/Vencord that referenced this pull request Jul 3, 2024
maddie480 added a commit to maddie480/Vencord that referenced this pull request Jul 3, 2024
@Jono997
Copy link
Author

Jono997 commented Jul 5, 2024

Well, uh, that's one way to solve the problem.

maddie480 added a commit to maddie480/Vencord that referenced this pull request Jul 5, 2024
maddie480 added a commit to maddie480/Vencord that referenced this pull request Jul 6, 2024
@maddie480
Copy link
Sponsor

maddie480 commented Jul 6, 2024

Uh yeah, that's just a fork I made for myself for developing a plugin 😅 And given that I rebase the branch to keep my Vencord up-to-date, I'm spamming quite a bit here, I'm going to remove the mention

(also, I didn't mean to mock anyone with that trollface message there, that's why I amended the commit 1 minute later but well, it was too late)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants