Skip to content

added ace perms for tablet#28

Open
WillFJS wants to merge 1 commit intoSonoran-Software:standalonefrom
WillFJS:standalone
Open

added ace perms for tablet#28
WillFJS wants to merge 1 commit intoSonoran-Software:standalonefrom
WillFJS:standalone

Conversation

@WillFJS
Copy link

@WillFJS WillFJS commented Feb 1, 2026

Description

Adds optional ace perm to check to see if client can use /radio tablet & /dispatchtablet

Motivation and Context

Some communities may want to restrict the use of the in-game radio tablet to dispatchers only and not open it to the public

How Has This Been Tested?

Yes, tested w/ 2 people in my test server

Contributor License Agreement

By submitting this pull request, you certify that you have read and agree to the project’s Contributor License Agreement (CLA) under the PolyForm Noncommercial 1.0.0 license.

I, WillFJS, hereby agree to license my contributions to radio_fivem under the PolyForm Noncommercial License 1.0.0.

(Replace <GitHub Handle> with your GitHub username.)

Checklist

Copy link
Member

@Jordan2139 Jordan2139 left a comment

Choose a reason for hiding this comment

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

Please update the permission check system to utilize the already existing permission check system found on line 604 of sv_main.lua

if IsPlayerAceAllowed then
local playerId = PlayerId()
local serverId = GetPlayerServerId(playerId)
return IsPlayerAceAllowed(playerId, 'sonoranradio.tablet') or IsPlayerAceAllowed(serverId, 'sonoranradio.tablet')
Copy link
Member

Choose a reason for hiding this comment

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

IsPlayerAceAllowed is not exposed to the client side and must be called from the server files. I would recommend sending the full user permissions in the event SonoranRadio::CheckPermissions found on line 604 of sv_main.lua

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.

2 participants