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

feat(Player): implement esx:setGroup event #1351

Merged
merged 2 commits into from
May 19, 2024
Merged

Conversation

tomiichx
Copy link
Contributor

@tomiichx tomiichx commented May 4, 2024

No description provided.

@Gellipapa
Copy link
Member

Hi! Thanks pr, can you explain some use cases when you would use this?

@Gellipapa Gellipapa changed the base branch from main to dev May 4, 2024 20:32
@tomiichx
Copy link
Contributor Author

tomiichx commented May 6, 2024

Hi! Thanks pr, can you explain some use cases when you would use this?

Hi!

I've had a few cases where I had to run loops to check for new admins or players who got their permissions set back to user. I usually cache all my results and then modify the table instead of fetching all players constantly. So one use case would be the admin chat, and a scoreboard for example. I have an admin count on the scoreboard, and if we add a new admin or remove one, I can't update my cached table without running a loop every minute or so...

I generally think this would be useful as esx:setJob is...

@Gellipapa
Copy link
Member

Hi! @tomiichx Thanks for this pr, i tested, it's works, merged.

@Gellipapa Gellipapa merged commit 66f8409 into esx-framework:dev May 19, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants