-
Notifications
You must be signed in to change notification settings - Fork 13
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
Setup mode #479
Comments
@ChrisSchinnerl @peterjan @alexfreska this is a feature I would like to discuss. We have this for the desktop apps, but configuring the apps on headless devices is still a friction point. |
That's interesting. Do you want to spin up the regular UI or a special page only for settings and submitting the config after which it gets written to disk and the actual UI and hostd get spin up? |
The way I understand: Pre setup:
Post setup:
Is this in line with your thinking? Do you both envision this is as only for first time config? Or could the user visit this page post setup and modify the config/trigger restart? |
That’s pretty much exactly the workflow I envisioned. A one time API and setup flow that disappears after it’s complete and writes a yaml config to disk. The regular UI will be redirected to a setup page since the user can’t login. We could add a config endpoint to the regular API to update the YAML, but it should not expose the wallet seed. |
Since serving the regular UI doesn't make sense during the setup stage, should we just serve the setup page on |
@alexfreska would creating a distinct UI overcomplicate the UI packaging and build process? Having one external UI package is already a bit of a pain. |
We can make the setup load on |
IMO if you do this on |
If the user has not configured a seed or API password,
hostd
should enter a setup mode with a single endpoint that allows the user to configure the yaml file via the web UI instead of failing to start. After the user has completed the initial setup, hostd should start normally without needing to be restarted.The text was updated successfully, but these errors were encountered: