search

LVPlayground / playground

The Las Venturas Playground gamemode
Other
32 stars 23 forks source link

Addition and Modification in OnPlayerCommandText.pwn #842

Open Specifer opened 3 years ago

Specifer commented 3 years ago

Change in /admins and addition of /vips.

Tested.

Change in /admins I have added a level Temp. Admin in /admins for players to differentiate full-time admins and part-time, Since temp admins operate at limited power. I felt this knowledge should be transparent to fellow players. Tested it thoroughly. And admins can see who temp'ed the person while players only see temp-admin. Also, added Developers to this list since it was long overdue? What do you think?

Addition of /vips

Just similar to /admins but shows also higher level (i.e) VIP, Developer, Administrator, and Manager.

FreakinsPL commented 3 years ago

My opinions 1) Make a separate cmd to see ingame developers (/devs)rather than including them in /admins coz they don't have admin rights.

2) I don't think so there's a reason of making /vips, why ppl wants to see who is vip and who is not? :)

@OttoRocket-LVP @Specifer

OttoRocket-LVP commented 3 years ago

Thank you for your modification as always!

Are you sure you fully tested this @Specifer? (With all levels mentioned?) I think there might be some minor issues.

Also please see inline comments.

Specifer commented 3 years ago

Yes Tested fully by myself. I'll also recheck once it is in staging with Freakins.

Specifer commented 3 years ago

Everything good?

Specifer commented 3 years ago

Dude holsje? Reply if this new commit has resolved the errors.

FreakinsPL commented 3 years ago

It would be good if we make this /vips cmd simple as possible.

Even if the players are administrators, managers or a developers. The /vips command should show only -VIP after the player name

example:-

Vip connected: Freakins -VIP Specifer -VIP Holsje -VIP Xilent -VIP