Skip to content
New issue

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

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

Already on GitHub? # to your account

Check legacy "NPC:" UUIDs in VaultEconomyProvider#hasAccount #4178

Merged
merged 2 commits into from
May 31, 2021

Conversation

JRoy
Copy link
Member

@JRoy JRoy commented May 27, 2021

awaiting confirmation from user who had this bug

@JRoy JRoy added the bug: confirmed Confirmed bugs in EssentialsX. label May 27, 2021
@mdcfe mdcfe added this to the 2.19.0 milestone May 31, 2021
@mdcfe mdcfe changed the title Fix VaultEconomyProvider#hasAccount sometimes returning false for NPCs Check legacy "NPC:" UUIDs in VaultEconomyProvider#hasAccount May 31, 2021
@mdcfe mdcfe enabled auto-merge (squash) May 31, 2021 15:12
@mdcfe mdcfe merged commit f8f1fac into EssentialsX:2.x May 31, 2021
@JRoy JRoy deleted the fix/sadge branch May 31, 2021 22:24
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug: confirmed Confirmed bugs in EssentialsX.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants