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

FOB & SITREP missing for players on dedicated servers #189

Merged
merged 2 commits into from
Sep 7, 2022
Merged

FOB & SITREP missing for players on dedicated servers #189

merged 2 commits into from
Sep 7, 2022

Conversation

CaiusAjiz
Copy link

@CaiusAjiz CaiusAjiz commented Aug 7, 2020

On dedis the SITREP and Request FOB radio functions are missing. Looks to be because BIS_fnc_addCommMenuItem is per client and not run (as far as I can see). Adding the BIS_fnc_addCommMenuItem's to the bottom of fn_clientInit.sqf works well enough for both initial and JIP players.

Tested on SP, MP and Dedicated server. No issues found as a result of this.

Fixes #175, #135 and an unreported issue with SITREP not being available. (plus a typo and missing semicolon)


This change is Reviewable

Caius Ajiz added 2 commits August 7, 2020 18:28
On a dedicated server, SITREP and Requesting a FOB never appears. This looks to be because BIS_fnc_addCommMenuItem is local, so needs to be called on the client, and i cannot for the life of me see if it already is. So i've just added it to the bottom of fn_clientInit.sqf.
Copy link

@jonneymendoza jonneymendoza left a comment

Choose a reason for hiding this comment

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

looks good

@CaiusAjiz
Copy link
Author

Anyone able to merge this PR? I've not got write to the repo.

Copy link
Contributor

@fritogotlayed fritogotlayed left a comment

Choose a reason for hiding this comment

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

Hey. Sorry about the lack of traction. I'm not really a maintainer here anymore. I'm not sure if the other maintainer is still active. Would you be interested in taking a maintenance role on the project? I can see about getting you the appropriate privileges if you would like.

@fritogotlayed fritogotlayed merged commit f46ac74 into DUWS-R-Team:master Sep 7, 2022
@CaiusAjiz
Copy link
Author

Hi there, it's ok, life happens. And it's been a long time since I've looked at anything SQF but I could quite happily pick it up again. I'm happy once I've got perms to sweep through the open issues and close some stuff off and get some clarity on what to work on in the github issues.

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.

No FOB option on dedicated servers
3 participants