-
Notifications
You must be signed in to change notification settings - Fork 164
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
[UI] New UI for fleets and instances #2236
Conversation
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.
@olgenn, thanks for the PR! I tested the new pages and they work as expected except for a few issues. I'll list them below.
Major:
-
The Instances list page fails to load if there are any pending instances. Console error:
React Router caught the following error during render TypeError: item.instance_type is null
.not-loaded-when-pending.mp4
-
The pagination seems to be off in the Instances list and Fleets list pages. If I open the second page and then return to the first page, it looks different than when I first opened it. If I then go through the pages again, it shows three pages, although at first there were only two.
pagination.mp4
Not major, can be finished after merging this PR:
- On the Fleets list and Fleet details pages, we won't need the
Backend
,Region
, andSpot
fields. ThePrice
field should be calculated as the sum of non-terminated instance prices. And we also need theInstances
field with a summary of non-terminated fleet instances. Please see the design document for a detailed specification of the desired fields. - We'd also like to show terminated fleets on the Fleets list page. Maybe add an "Active fleets" trigger?
- The document also requests a "Delete" button on the Fleet details page.
- The document also requests links to fleet details in the "Fleet" field of the Instances list page.
- Can we also add links to project settings in the "Project" field of the Instances list page?
- If the instance costs 0, its price on the Instances list page is shown without a dollar sign. Can we add the dollar sign?
Fixes after review
Fixes after review
Fixes after review
Fixes after review
Fixes after review
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.
👍
Fixes after review
Close #2133