Skip to content
This repository has been archived by the owner on Dec 6, 2024. It is now read-only.

feat: display Configuration Name and Instance Type on Workspace details card #669

Merged
merged 16 commits into from
Aug 24, 2021

Conversation

maghirardelli
Copy link
Contributor

Issue #, if available: GALI-284

Description of changes: Added two new rows at the bottom of the details table for the Workspace cards that display the Configuration Name and Instance Type of the workspace for researchers to better understand the capacity of the workspace:
Screen Shot 2021-08-18 at 4 17 34 PM

Checklist:

  • Have you successfully deployed to an AWS account with your changes?
  • Have you successfully tested with your changes locally?

AS review ticket id:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@codecov
Copy link

codecov bot commented Aug 18, 2021

Codecov Report

Merging #669 (7f6b6d0) into develop (2c5bb1a) will decrease coverage by 0.09%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #669      +/-   ##
===========================================
- Coverage    49.20%   49.11%   -0.10%     
===========================================
  Files          246      248       +2     
  Lines        12665    12759      +94     
  Branches      2023     2041      +18     
===========================================
+ Hits          6232     6266      +34     
- Misses        5629     5683      +54     
- Partials       804      810       +6     
Impacted Files Coverage Δ
src/parts/environments-sc/ScEnvironmentCard.js 40.35% <0.00%> (ø)
...c/parts/environments-sc/parts/ScEnvironmentCost.js 3.44% <0.00%> (ø)
src/models/environment-types/EnvTypeConfig.js 60.00% <0.00%> (+43.33%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2c5bb1a...7f6b6d0. Read the comment docs.

@maghirardelli maghirardelli changed the title feat: display Workspace Id and Instance Type on Workspace details card feat: display Configuration Name and Instance Type on Workspace details card Aug 18, 2021
Copy link
Contributor

@SanketD92 SanketD92 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unit and E2E tests needed.

@Bingjiling
Copy link
Contributor

We need to add these two fields to the env type config cards for selection before launching workspace as well.

@github-actions github-actions bot added size/m and removed size/s labels Aug 19, 2021
@github-actions github-actions bot added size/s and removed size/m labels Aug 19, 2021
@maghirardelli
Copy link
Contributor Author

Display for the select configuration cards:
Screen Shot 2021-08-19 at 1 54 38 PM

@github-actions github-actions bot added size/m and removed size/s labels Aug 23, 2021
Copy link
Contributor

@nguyen102 nguyen102 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Looks good!

@maghirardelli maghirardelli merged commit f0fa819 into develop Aug 24, 2021
@maghirardelli maghirardelli deleted the workspacedetails branch August 24, 2021 19:05
jxuamazon pushed a commit to jxuamazon/service-workbench-on-aws that referenced this pull request Feb 15, 2022
…ls card (awslabs#669)

feat: display Configuration Name and Instance Type on Workspace details card and when selecting configuration to create a new workspace
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants