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

Side channel in the PvP composition isn't re-enabled #159

Open
freghar opened this issue Mar 14, 2021 · 1 comment
Open

Side channel in the PvP composition isn't re-enabled #159

freghar opened this issue Mar 14, 2021 · 1 comment
Labels
Bug Existing feature that doesn't work as intended Default Modules (PvP) Issues regarding the default pvp mission modules

Comments

@freghar
Copy link
Collaborator

freghar commented Mar 14, 2021

This is because it's disabled by an Arma Additions CBA feature and the Execute Code then doesn't re-enable it, it only tries to disable others.

Change its code to

/* Chat channels for PvP - make sure Side is enabled */

addMissionEventHandler ["PreloadFinished", {
    0 = [] spawn {
        /* make doubly sure this runs after CBA settings */
        sleep 1;
        1 enableChannel true;
    };
}];

to actually re-enable the Side channel. Set Environment to Unscheduled otherwise it might not work.

Note that (to make this reliable for JIP, if we ever run PvP with JIP) the channel is re-enabled 1 second after entering the game, so any planning has to be done after the briefing screen.

@JamesTheClarke JamesTheClarke added Bug Existing feature that doesn't work as intended Default Modules (PvP) Issues regarding the default pvp mission modules labels Mar 14, 2021
@JamesTheClarke JamesTheClarke added this to the v2.0.1 milestone Mar 14, 2021
@JamesTheClarke
Copy link
Contributor

related to isue #151

JamesTheClarke added a commit that referenced this issue Mar 18, 2021
- added comment marker to warn about the need for an inhibitor module when updating this composition
- updated version number to v2.0.1
- updated LR channel 5 name from "HQ" to "HQ Net" to match the others
- removed one duplicate execute code module determining the chat settings, but left one standing for now. May be removed later due to issue #159
- removed required addons from the header.sqe file
JamesTheClarke added a commit that referenced this issue Mar 18, 2021
- added comment marker to warn about the need for an inhibitor module when updating this composition
- updated version number to v2.0.1
- updated LR channel 5 name from "HQ" to "HQ Net" to match the others
- removed one duplicate execute code module determining the chat settings, but left one standing for now. May be removed later due to issue #159
- removed required addons from the header.sqe file
@JamesTheClarke JamesTheClarke removed this from the v2.0.1 milestone Mar 19, 2021
@JamesTheClarke JamesTheClarke added this to the v3.0.1 milestone May 31, 2021
@JamesTheClarke JamesTheClarke removed this from the v3.0.1 milestone Nov 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Existing feature that doesn't work as intended Default Modules (PvP) Issues regarding the default pvp mission modules
Projects
None yet
Development

No branches or pull requests

2 participants