Qbox-project / qbx_adminmenu

GNU General Public License v3.0
2 stars 33 forks source link

use txadmin noclip #49

Closed DevX32 closed 8 months ago

DevX32 commented 8 months ago

Description

Checklist

DevX32 commented 8 months ago

need to fix the noclip command searching for the txadmin event

DevX32 commented 8 months ago

@Manason

solareon commented 8 months ago

So the issue I see with this one is does it work if you don't have txadmin perms to no clip? Some server owners/admins may not want to give txadmin permissions to folks hence the desire for an alternative noclip

DevX32 commented 8 months ago

So the issue I see with this one is does it work if you don't have txadmin perms to no clip? Some server owners/admins may not want to give txadmin permissions to folks hence the desire for an alternative noclip

but server owners can give only noclip access though i asked manason he said its good though

DevX32 commented 8 months ago

i dont think there is diffrence between the tx noclip and normal admin one though

mafewtm commented 8 months ago

If this required server owners to add people to the cfg just for noclip access, I think we should stick with what we have already

Manason commented 8 months ago

Needs to be tested in a scenario where a non-owner admin without txAdmin access can noclip

Manason commented 8 months ago

If this required server owners to add people to the cfg just for noclip access, I think we should stick with what we have already

I assume you are talking about ACE permissions, but that can be done at runtime using ox_lib.

BerkieBb commented 8 months ago

If this required server owners to add people to the cfg just for noclip access, I think we should stick with what we have already

That doesn't make sense as we use the cfg for all our permissions too

mafewtm commented 8 months ago

If this required server owners to add people to the cfg just for noclip access, I think we should stick with what we have already

That doesn't make sense as we use the cfg for all our permissions too

It makes perfect sense. Why have to set another permission for someone when you can just set them as mod and it deals with noclip already.

BerkieBb commented 8 months ago

If this required server owners to add people to the cfg just for noclip access, I think we should stick with what we have already

That doesn't make sense as we use the cfg for all our permissions too

It makes perfect sense. Why have to set another permission for someone when you can just set them as mod and it deals with noclip already.

We can incorporate that in the mod group

mafewtm commented 8 months ago

We should just maintain our own version of noclip