Skip to content

Conversation

@coyotan
Copy link
Contributor

@coyotan coyotan commented Oct 28, 2025

Solves #57 by adding environment variables to configure VONDisableUI, VONDisableDirectSpeechUI, and VONCanTransmitCrossFaction.

Add options for customizing VONDisableUI, VONDisableDirectSpeechUI, and VONCanTransmitCrossFaction in the environment.
@BrettMayson BrettMayson requested a review from Copilot October 28, 2025 15:54
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for configuring three Voice Over Network (VON) game properties through environment variables: VONDisableUI, VONDisableDirectSpeechUI, and VONCanTransmitCrossFaction. These settings allow server administrators to control voice chat UI visibility and cross-faction communication capabilities.

Key changes:

  • Added three new environment variables for VON configuration with default values of false
  • Implemented logic to read these environment variables and apply them to the game configuration
  • Updated default JSON configuration to include the new VON properties

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
launch.py Added conditional logic to read VON-related environment variables and populate gameProperties configuration
docker_default.json Added default VON property values to the base game configuration
Dockerfile Defined three new environment variables for VON settings with false defaults

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@BrettMayson BrettMayson merged commit 642e722 into acemod:main Oct 28, 2025
1 check passed
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.

2 participants