-
Notifications
You must be signed in to change notification settings - Fork 61
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
Prompt the user to select a project #3847
Prompt the user to select a project #3847
Conversation
Here is a demo of the workflow: create-or-select-project-prompt.mp4 |
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.
Looks good to me. Thanks!
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.
LGTM
Checking the test failure before merging... |
@datho7561 It only looks like the following test failure is related to the change:
|
When trying to run "Start Dev", "Manually Start Dev", "Create" (to create resources from a YAML file), or "Delete" (to delete resources from a YAML file), if the current project doesn't exist, display a notification initicating that a project needs to be selected or created, with a button that opens the wizard to do this. Fixes redhat-developer#2581 Signed-off-by: David Thompson <davthomp@redhat.com>
4df77b0
c198d84
to
4df77b0
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #3847 +/- ##
==========================================
+ Coverage 32.47% 32.58% +0.11%
==========================================
Files 85 85
Lines 6418 6441 +23
Branches 1333 1337 +4
==========================================
+ Hits 2084 2099 +15
- Misses 4334 4342 +8 ☔ View full report in Codecov by Sentry. |
I rewrote the test to reflect the new behaviour. Are you okay if I merge this @vrubezhny @msivasubramaniaan ? |
@datho7561 +1 for merging the PR - All the tests successfully passed now |
When trying to run "Start Dev", "Manually Start Dev", "Create" (to create resources from a YAML file), or "Delete" (to delete resources from a YAML file), if the current project doesn't exist, display a notification initicating that a project needs to be selected or created, with a button that opens the wizard to do this.
Fixes #2581
Signed-off-by: David Thompson davthomp@redhat.com