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

Core - Add last group hint when player re-jips in #626

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

PabstMirror
Copy link
Contributor

Adds hint for player's last group when the jip in to make it faster and easier to get them back to where they should be
Remove time from admin log and moves it to tooltip

@kil0byt3
Copy link
Member

Remove time from admin log and moves it to tooltip

amazing, both sound great

Copy link
Contributor

@TheCandianVendingMachine TheCandianVendingMachine left a comment

Choose a reason for hiding this comment

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

imo the civilian case needs to be handled, we use them often enough for this to be annoying

@@ -40,6 +40,13 @@ private _return = if (_vehicle == _leader) then {
["Teleporting to position", "No empty position for teleport"] select _success
}
} else {
[{ // hint if TP-to-Vic fails
Copy link
Contributor

Choose a reason for hiding this comment

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

(nitpick)
niche case, but if someone is fucking around in the start zone and enter a vehicle they won't have this pop

or if we add a shitty addon that for some reason makes this return an object

params ["_player", "_profileName"];
private _msg = if (_profileName in GVAR(playerMap)) then {
(GVAR(playerMap) get _profileName) params ["_lastSide", "_lastGroup"];
private _sideText = ["?", "B", "O", "I"] select ([blufor, opfor, independent] find _lastSide);
Copy link
Contributor

Choose a reason for hiding this comment

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

civilian handling doesn't work, it will default to I since find == -1

Copy link
Contributor Author

Choose a reason for hiding this comment

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

opps, I think there should have been a subtraction here, I'll just swap to a switch

format ["has JIPed"];
};
INFO_2("playerConnected: %1: [%2]",_profileName,_msg);
if (CBA_missionTime < 60) exitWith {};
Copy link
Contributor

Choose a reason for hiding this comment

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

future:
can probably set this timer to be the safe-start length as specified in the mission parameters option

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.

3 participants