-
Notifications
You must be signed in to change notification settings - Fork 1
Update Chronos references in UI #124
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
Conversation
✅ Deploy Preview for subspacefaucet canceled.
|
|
The setup of the web faucet is a little bit messy, and I haven't found a smart way to test this locally. This PR is very likely to fix the remaining issues with the web faucet, but we will only be able to confirm once it is merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I personally find the web UI/UX to be quite unpleasant but haven't been able to give thought to how to improve. CI is failing on a missing test file and I think any new env variables should be added to the .env.sample file. Other than those, I'm ok with testing this on merge since it already doesn't work 🙂 .
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Closed as replaced by #125 |
This PR adds the necessary visual elements to support requesting tokens on Chronos via web interface.
Most of the setup was moved to the env file, which should make next deployments much simpler.