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

[Feature]: Allow the default channel to be set by the administrator #5

Closed
hickey opened this issue Sep 24, 2023 · 1 comment · Fixed by #19
Closed

[Feature]: Allow the default channel to be set by the administrator #5

hickey opened this issue Sep 24, 2023 · 1 comment · Fixed by #19
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@hickey
Copy link
Owner

hickey commented Sep 24, 2023

Contact Details

No response

Enhancement Type

Graphical interface

What is your idea or what can be improved?

Original request for this feature was stated in kn6plv/meshchat#23.

KD4WLE suggested the solution of allowing the administrator to set the default channel that a user is placed into when they login to MeshChat. This avoids people being overwelm by the Everything channel.

@hickey
Copy link
Owner Author

hickey commented Nov 24, 2023

#19 has been progressing, but there are some data synchronization problems. A good amount of problems have been fixed with a Javascript promise to properly load the configuration data from the config API. Now the problem is that the channel select boxes do not get populated until process_messages() gets called well after loading the configuration.

I also fore see future problems with processing messages in the current fashion as the message synchronization gets improved. For the most part the only real way to handle the problem is to create a message class that can handle the DB manipulation and provide a consistent API for the rest of the code to utilize any time a message is being touched.

@hickey hickey closed this as completed in #19 Mar 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant