Skip to content

Add "go back" possibility to cluster login quick picks #3357#3359

Merged
datho7561 merged 1 commit intoredhat-developer:mainfrom
vrubezhny:fix-cluster-login-quick-pick-add-go-back-possibility
Oct 2, 2023
Merged

Add "go back" possibility to cluster login quick picks #3357#3359
datho7561 merged 1 commit intoredhat-developer:mainfrom
vrubezhny:fix-cluster-login-quick-pick-add-go-back-possibility

Conversation

@vrubezhny
Copy link
Copy Markdown
Contributor

Fixes: #3357

@codecov
Copy link
Copy Markdown

codecov bot commented Sep 27, 2023

Codecov Report

Attention: 222 lines in your changes are missing coverage. Please review.

Comparison is base (c07f159) 0.00% compared to head (161a45f) 36.02%.

Additional details and impacted files
@@            Coverage Diff            @@
##           main    #3359       +/-   ##
=========================================
+ Coverage      0   36.02%   +36.02%     
=========================================
  Files         0       77       +77     
  Lines         0     5549     +5549     
  Branches      0     1068     +1068     
=========================================
+ Hits          0     1999     +1999     
- Misses        0     3550     +3550     
Files Coverage Δ
src/openshift/cluster.ts 7.00% <0.89%> (ø)

... and 76 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@vrubezhny
Copy link
Copy Markdown
Contributor Author

How Back buttons work for navigation through the Cluster Login Quick Picks:

screencast-cluster-login-back-buttons

datho7561
datho7561 previously approved these changes Sep 29, 2023
Copy link
Copy Markdown
Contributor

@datho7561 datho7561 left a comment

Choose a reason for hiding this comment

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

Seems to work well. Found some dead code

Comment thread src/openshift/cluster.ts Outdated
@vrubezhny vrubezhny force-pushed the fix-cluster-login-quick-pick-add-go-back-possibility branch from 2a566fc to 161a45f Compare September 29, 2023 21:17
@vrubezhny vrubezhny requested a review from datho7561 September 29, 2023 21:21
Copy link
Copy Markdown
Contributor

@datho7561 datho7561 left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks, Victor!

@datho7561 datho7561 merged commit 93a7c4a into redhat-developer:main Oct 2, 2023
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.

Add "go back" possibility to cluster login quick picks

2 participants