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

[WIP] Add the manager UI #59

Closed
wants to merge 34 commits into from
Closed

[WIP] Add the manager UI #59

wants to merge 34 commits into from

Conversation

Senyoret1
Copy link
Contributor

Changes:

  • This PR includes the changes made in [WIP] Add the manager UI #23, but targets the master branch instead of mainnet-milestone2. It also has some minor changes nedded for making the manager work with the master branch.

How to test this PR:
Follow the instructions in static/skywire-manager-src/README.md

@Senyoret1 Senyoret1 mentioned this pull request Nov 9, 2019
@Senyoret1 Senyoret1 changed the title Add the manager UI [WIP] Add the manager UI Nov 11, 2019
@Senyoret1
Copy link
Contributor Author

The lastest commit updates the Angular version used by the manager from v6.1 to v8. Multiple changes were made to make the manager work well with v8 and to prepare it to work well with some changes that are expected for v9.

@Senyoret1
Copy link
Contributor Author

The lastest commit adds compatibility with the lastest changes made to the hypervidor API. It includes the following changes:

  • The online param returned by the API for knowing if a node is online is now being used.
  • It is now possible to use the UI to set if an app should be automatically started or not.

@Senyoret1
Copy link
Contributor Author

The lastest commit includes the following changes:

  • A big part of the code related to the left menu has been changed, to make it compatible with a responsive design.
  • Now the left menu is hidden and a menu bar is shown at the top when the screen is too narrow.

This is a start for making the design work well on small screens, as it currently looks good only if the browser window is 900px wide or more.

@Senyoret1
Copy link
Contributor Author

The lastest commit includes the following changes:

  • A dedicated control for showing truncated text was added, to avoid having to use CopyToClipboardTextComponent in hacky ways.
  • The scroll position now works better after navigating between pages.
  • The option for copying the node key in the nodes list was removed, as it was very confusing to use.
  • Several warnings reported by tslint were solved.
  • A small problem with a tooltip was solved.
  • A problem that made an error msg to show an unused var was fixed.

@Senyoret1
Copy link
Contributor Author

The lastest commit changes the design of the nodes list to make it responsive. It includes the base styles that will be use to improve the other lists.

@Senyoret1
Copy link
Contributor Author

The lastest commit includes the following changes:

  • The tab bar now works well on small screens.
  • The hover effect of the buttons is now easier to see, as after updating to Angular 8 it became very subtle.
  • Various additional fixes for the nodes list.
  • A problem with the left menu in the settings page was solved.

@Senyoret1
Copy link
Contributor Author

The lastest commit includes the following changes:

  • Now the modal windows work well in small screens.
  • The size of the modal windows was standarized and the code for openning them is easier to mantain.
  • Several small improvements for multiple modal windows.

@Senyoret1
Copy link
Contributor Author

The last 2 commits include the following changes:

  • Now the info bar, at the right side of the nodes page, works well on small screens.
  • The disbled state of the responsive menu is now easier to identify in the UI.
  • The language selection modal window now works better in small screens.
  • Other small improvements.

@Senyoret1
Copy link
Contributor Author

The last 2 commits include the following changes:

  • Now the sorting options work on small screens.
  • A small improvement for the code.

@Senyoret1
Copy link
Contributor Author

The last 2 commits include the following changes:

@Senyoret1
Copy link
Contributor Author

The lastest commit includes the following changes:

  • The internal tables now work well on small screens.
  • The alert that prevented using the manager on small windows was removed.
  • Several improvements for the design and style files.
  • Several small fixes.

@Senyoret1
Copy link
Contributor Author

The lastest commit includes the following changes:

  • Improvements for displaying the info bar at the right if the window is less than 1300px wide.
  • Improvements for the tabs bar.
  • The texts that can be copied by clicking them now show a small icon. Also, now the text can be selected normally using the mouse.
  • More texts can now be copied with a mouse click.
  • Some improvements to avoid potential future problems with the layout.
  • Improvements for the UX in the lists.
  • Other fixes and improvements.

@Senyoret1
Copy link
Contributor Author

The lastest commit includes the changes made when cleaning and commenting the rest of the code. It includes the rest of the files and much more changes than the previous commit, so now most of the old code has been removed and several parts are much simpler to read and mantain, but some old code is still present, waiting to see if it could be useful when implenting the features that are still missing. Also, a list with other small problems was created while cleaning the code.

@Senyoret1
Copy link
Contributor Author

The lastest commit includes the following changes:

  • Several improvements for the code quality.
  • Several small bugs were fixed.
  • Improvements for the app list.
  • Improvements for the app's internal routing management.
  • The loops list was removed from the routing page.

@Senyoret1
Copy link
Contributor Author

The lastest commit includes the following changes:

  • A small problem in the navigation system related to the login page was solved.
  • The table buttons are now more consistent with the design of the rest of the app.
  • AutoScalePipe is now compatible with multiple data measurements.
  • ApiService now uses well deffined objects for the params.
  • A problem with how the tables are sorted and navigation was solved.
  • Some other snall fixes.

@Senyoret1
Copy link
Contributor Author

The lastest commit includes the following changes:

  • The UI/UX of the paginators was improved. Now the total amount of pages is displayed and it is possible to open a modal window to navigate to a specific page faster that by using the navigation buttons.
  • Improvements were made in some code sections that were not always showing an adecuate error msg in the UI.
  • The inactive node options were hidden.
  • The default apps bar was hidden, as it is not working yet.
  • Small additional improvements.

@Senyoret1
Copy link
Contributor Author

The lastest commit includes the following changes:

  • The base translation file was cleaned.
  • The Spanish translation file was created and now the Spanish option appears in production builds.
  • The return button in the node details page was improved.
  • The app logo was updated.
  • Other small fixes.

@Senyoret1 Senyoret1 mentioned this pull request Dec 8, 2019
23 tasks
@Senyoret1
Copy link
Contributor Author

The lastest commit fixes a small error in the README.md file

@Senyoret1
Copy link
Contributor Author

The lastest commit includes the following changes:

  • The front-end now works using the API of the milestone2 branch, because that branch is going to be merged soon. It does not work with the current master branch.
  • The option for restarting the visors was added.
  • Some fixes for the transport details modal window.
  • Small changes in the password management, to adapt it to the milestone2 branch.
  • Some other small improvements.

@Senyoret1
Copy link
Contributor Author

The last 2 commits include the following changes:

  • The route details modal window now includes several details that were not being shown and has compatibility with the "Intermediary forward" rule type.
  • Now the dmsgpty-ui terminal from the /pty route of the hypervisor is used when accessing the manager with an http URL, due to problems with https. When using a https url for opening the manager, the old terminal, which works with the /visors/{pk}/exec API endpoint, is used.

@Senyoret1
Copy link
Contributor Author

The lastest commit includes the following changes:

  • An option for updating the visors was added. The option only request confirmation from the user and uses the API endpoint for updating the node, so there is no info about if there are updates availables.

@Senyoret1
Copy link
Contributor Author

The lastest commit includes the following changes:

  • Part of the code for error management of the desktop wallet was adapted and added to the manager, as previously the manager was depending a lot on assumptions which were not really good for displaying accurate error messages. Some parts are now showing the error messages using the new information obtained with the new procedures, for giving more accurate information in cases in which it is needed (like when the updater fails), and other parts were not changed, to avoid making too many changes with potential errors before the release.

@Senyoret1
Copy link
Contributor Author

The lastest commit includes the following changes:

  • The proxy for the Angular dev server was updated to use https connections for the API calls, as needed in the lastest version of the milestone2 branch.

@Senyoret1
Copy link
Contributor Author

The lastest commits include the following changes:

  • Now the user can select which terminal to open: the new dmsgpty-ui terminal on a new tab or the basic terminal inside the manager. The basic terminal works in some cases dmsgpty-ui does not and on mobile devices.
  • The Spanish translation was updated.
  • Various small bugs were fixed.

@Senyoret1 Senyoret1 mentioned this pull request Mar 9, 2020
@jdknives jdknives closed this Mar 9, 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