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

[SUGGESTION] Add command for permanent permissions #888

Open
SharkyFP opened this issue Dec 8, 2022 · 1 comment
Open

[SUGGESTION] Add command for permanent permissions #888

SharkyFP opened this issue Dec 8, 2022 · 1 comment
Assignees

Comments

@SharkyFP
Copy link

SharkyFP commented Dec 8, 2022

The problem

Since we have had ace permissions added as the new permission system there hasn't been a way of saving players permissions if he isn't added manually in the server.cfg. So it would be nice if there was a way of saving/printing add_principal inside server.cfg when using command /addpermission if it could be possible to add lines thru a script inside server.cfg

Ideal solution

So when using /addpermission [ID] [identifier] [permission.name]
It saves the params inside server.cfg if it even possible to access server.cfg thru a script.

Alternative solutions

If it isn't possible to access the server.cfg thru a script to write inside it then have a different sile such as permissions.cfg or whatever extention it may be and use the file to check whenever the QBCore.Functions.HasPermission is checked.

Additional context

No response

@SharkyFP SharkyFP added the enhancement New feature or request label Dec 8, 2022
@Z3rio Z3rio changed the title [SUGGESTION] [SUGGESTION] Add command for permanent permissions Dec 9, 2022
@GhzGarage GhzGarage added suggestion and removed enhancement New feature or request labels Jun 1, 2023
@GhzGarage GhzGarage self-assigned this Jun 1, 2023
@martin-lechene
Copy link

Never rewrite in .cfg file when the server starter.
But good idea and it's work by TxAdmin.

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

Successfully merging a pull request may close this issue.

3 participants