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

Add support for meshchat_local.lua #47

Merged
merged 1 commit into from
Mar 3, 2024
Merged

Add support for meshchat_local.lua #47

merged 1 commit into from
Mar 3, 2024

Conversation

hickey
Copy link
Owner

@hickey hickey commented Mar 3, 2024

Package upgrades do not update the meshchatconfig.lua because it is tagged as a configuration file. This prevents settings from being lost, but makes it difficult to add new settings in future releases. This change establishes the meshchat_local.lua file for local administrators to configure there MeshChat.

During installation of a package if a meshchat_local.lua is not found, then a copy of meshchatconfig.lua is copied and sanitized of the app_version and protocol_version settings.

@hickey
Copy link
Owner Author

hickey commented Mar 3, 2024

Local testing complete. Version 2.11.2 was installed and then upgraded with the package from this branch. The result created a meshchat_local.lua and it was created without any of the version variables.

Package upgrades do not update the `meshchatconfig.lua` because
it is tagged as a configuration file. This prevents settings
from being lost, but makes it difficult to add new settings
in future releases. This change establishes the `meshchat_local.lua`
file for local administrators to configure there MeshChat.

During installation of a package if a `meshchat_local.lua` is not
found, then a copy of `meshchatconfig.lua` is copied and
sanitized of the `app_version` and `protocol_version` settings.

Signed-off-by: Gerard Hickey <hickey@kinetic-compute.com>
@hickey hickey merged commit b51bb16 into master Mar 3, 2024
3 checks passed
@hickey hickey deleted the local-config branch March 3, 2024 04:33
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.

None yet

1 participant