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

Improvements for the updater UI #477

Merged
merged 3 commits into from
Aug 17, 2020
Merged

Improvements for the updater UI #477

merged 3 commits into from
Aug 17, 2020

Conversation

Senyoret1
Copy link
Contributor

Did you run make format && make check?
The go code was not changed. npm run lint and npm run build were used.

Changes:

  • An option for updating the hypervisor was added to the menu in the visor list.
  • An option for configuring the updater was added to the settings page. For accessing it, you must click the "Show updater settings" link at the bottom of the page.

How to test this PR:
Test the hypervisor updater using the option in the visor list. Test the updater settings using the form at the bottom of the settings page.

Note 1: #476 must be solved for the hypervisor update procedure to work totally well.

Note 2: the control for updating the hypervisor has a lot of repeated code from the control for updating a visor. This was made to avoid overcomplicating the control user for updating the visors, as the operation for updating the hypervisor will not be needed after some time.

@nkryuchkov nkryuchkov merged commit b095f02 into skycoin:develop Aug 17, 2020
@nkryuchkov nkryuchkov mentioned this pull request Aug 17, 2020
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.

2 participants