Qbox-project / qbx_management

GNU General Public License v3.0
5 stars 20 forks source link

Confirmation before hiring #38

Open Manason opened 6 months ago

Manason commented 6 months ago

The problem

A boss can force a nearby player into a gang/job

Ideal solution

use ox_lib to have the player confirm they want to be recruited/hired

Alternative solutions

Have them enter a confirmation command in chat like /confirm or /yes. This would prevent any abuse of forcing a popup on a players screen

Additional context

No response

mafewtm commented 6 months ago

I don't really think this is needed tbh

Manason commented 6 months ago

It doesn't need to be a release blocker, but it would be nice to have. Going to leave it open.

solareon commented 6 months ago

In qbx_vehiclesales there is a simple confirmation snippet that could be reused here.

mafewtm commented 6 months ago

I personally dont think we need to have players approve or deny employement as its in one location and its not something someone would usually abuse, but if you guys think it should be added I can do that

SKITTLE6969 commented 3 months ago

hell nahhh there will be no point having the fire option cuz the player can still have the job after decline and cause alot of issues @_@

alberttheprince commented 2 months ago

Depending on how far the user sets the hiring distance, this can be used accidentally to hire the wrong person. Putting in a confirmation screen allows the user to stop themselves being wrongly hired.