-
Notifications
You must be signed in to change notification settings - Fork 45
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
UI2024: Enable user configurations in Network Settings #106
Comments
This would definitely simplify the usage of custom RVS. But I'm not sure if it justifies the effort for the implementation. |
As always - thanks ,for your considerations of my ideas. This and the two other remaining issues are actually my last suggestions, as it seems I cannot come up with any more potential improvements to this great software.There have been a steep development his past weeks and I am quite excited at how it has turned out! Some context regarding this feature request is that I have setup a new permanent public RVS. This is my donation to the project and its users. You can test it by using
In the coming days I will look at the new web pages in Swedish and provide suggested adjustments to the wording (if needed), |
Wow, thank you! Concerning the To cut a long story short: I'll start implementing the integration into the UI as proposed, but will use the existing properties files to store the settings. |
Excellent! As you have understood already, I do not know enough programming. Your suggestion is a perfect solution to getting it right! |
Looking great! |
Q: the default box looks like it is editable. Just to clarify my suggestion, just in case: I think it should be "read only" so the user always can revert back to it if needed. Perhaps the default could be overwritten in Or perhaps Just a note, you understand all this so much better 👍 |
No, the default is "read only". As it's implemented now, all functionality from the The consequence of this is that the configuration from the yamls is (always) read and evaluated on start up. I'd prefer to keep those two configuration paths as separated as possible, because it could quickly become confusing. The primary use case for the yamls would be "vendored" instances, which should use a custom RVS by default. I didn't implement the "Do not use Token Server" option, because I thought that one can simply not use the feature, if he/she/it doesn't want to. |
Thanks for the good explanation, makes a lot of sense ❤️ |
I think your clear thoughts on "vendored" instances would be great to have in the descriptive section for the Here is my suggestion (to save you the pain of updating the documentation) Quick start Configuring Dayon! using configuration file This makes it possible to override user settings since the configuration from the YAML is always read and evaluated on start up. Users can make changes inside the application, but these only last until the next start if a YAML configuration is in place. The primary use case for YAML configurations would for organizations that want to use a self-hosted token server instead of the public token server, or set specific host details and make assisted autoconnecting when started. The structure of the assisted YAML is extremely simple:
The assistant also has a YAML configuration file.
The YAML file(s) can be saved under the name assisted.yaml and assistant.yaml either in the Dayon! home directory, in the user directory, or in the same directory as the .jar, resp. .exe file. This order also corresponds to the priority with which they are taken into account in the case of multiple configurations exist. (first one wins) ...and move the section "Automatic connection of the assisted" to the bottom of the page and rephrase it as follows: Automatic connection of the assisted via command line
If the assisted is started with these example parameters, it connects directly to the host "example.com" and port "4242". |
Idea: perhaps add a new short section at the bottom of How to run a custom rendez-vous server (RVS) Public token serversBelow you find a list of available public token servers that are free to use. Check the availability of a token server by clicking on its link. If it shows the right version number it is active. |
Thanks again, I'll make those suggested changes as soon as I get to it. |
I like it! Perhaps Quickstart could be reorganized (and split) as follows, as this would not require any new texts or images: Note: the numbering etc. below is only to give you readability, I imagine regular headline formats etc. Preparing & Connecting
section "Now you can ask the assisted to connect"
section "You're now connected and monitoring the remote computer." add new sentence: "Note: If you are unable to connect by following the instructions above, you should check that you have the right
section "Statistics Counters" moved from Support Page Suggested new page: Settings (placed between Quick Start and Download in the Top Menu) Settings & Configurations
To keep the number of web pages down, simply delete the web page Feedback and adjust the opening statement on the web page Support as follows (if you like it) SupportYour feedback is more than welcome! Post your question or open a new issue on GitHub. |
UI2024: connection settings I just realized that since Perhaps when starting the connection setting function in the app, it should check if it finds any If it does, it would show a pop up window stating something like "Warning: a configuration file has been found. Any changes made will only last until the next start of the application." (I believe this would reduce frustration for users that have not read those YAML sections online before using the app.) |
Well, I started restructuring the quickstart and settings pages. But it will take some time until it's done. |
Good that they get a rework then. Thanks for your efforts! |
UI extensions is included in release v14.0.0, the restructuring of the documentation is ongoing. |
Is your feature request related to a problem? Please describe.
The use of
assistant.yaml
andassisted.yaml
is a simple and elegant solution to user configurations. However, today it requires the user to download the files from GitHub, place them in an appropriate folder and edit them separately from Dayon!Describe the solution you'd like
A simple interface to edit contents of
yaml files
. It should enable users to easily revert to default token server and to enable/disable autoconnect.Possible approach in three steps:
Include the two
yaml files
in the installation package, in folder.dayon
(easy to access for all OS), instead of users downloading them separately from GitHubAssisted UI:
assisted.yaml
if pressing OK.assistant.yaml
if pressing OK.Note: to reduce clutter/fewer buttons for both users, the suggested option "do not use token server" would be used. It hides the token button (for Assistant - context Connection) and the token tab when connecting (for Assisted - after pressing "play button").
Describe alternatives you've considered
Additional context
The text was updated successfully, but these errors were encountered: