Skip to content

fix(es_extended/server/functions.lua): IsPlayerAdmin group fix#1618

Merged
Kenshiin13 merged 13 commits into
esx-framework:devfrom
iSentrie:patch-12
Mar 9, 2025
Merged

fix(es_extended/server/functions.lua): IsPlayerAdmin group fix#1618
Kenshiin13 merged 13 commits into
esx-framework:devfrom
iSentrie:patch-12

Conversation

@iSentrie

@iSentrie iSentrie commented Mar 2, 2025

Copy link
Copy Markdown
Contributor

Description

Function failed to properly detect is player admin or not, while GetPlayerFromId function was receiving a string instead of number.

PR Checklist

  • My commit messages and PR title follow the Conventional Commits standard.
  • My changes have been tested locally and function as expected.
  • My PR does not introduce any breaking changes.
  • I have provided a clear explanation of what my PR does, including the reasoning behind the changes and any relevant context.

@github-project-automation github-project-automation Bot moved this to 🆕 To be Discussed in ESX Roadmap Mar 2, 2025
@iSentrie iSentrie changed the title fix(functions.lua): IsPlayerAdmin group fix fix(es_extended/server/functions.lua): IsPlayerAdmin group fix Mar 9, 2025
@Kenshiin13 Kenshiin13 changed the base branch from main to dev March 9, 2025 21:59

@Kenshiin13 Kenshiin13 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@github-project-automation github-project-automation Bot moved this from 🆕 To be Discussed to Ready for Next Update in ESX Roadmap Mar 9, 2025
@Kenshiin13 Kenshiin13 merged commit 4627d8a into esx-framework:dev Mar 9, 2025
@iSentrie iSentrie deleted the patch-12 branch March 10, 2025 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Ready for Next Update

Development

Successfully merging this pull request may close these issues.

4 participants