-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
feat: Add/Delete support in Sources tab for multi-source apps (#18047) #18804
feat: Add/Delete support in Sources tab for multi-source apps (#18047) #18804
Conversation
❌ Preview Environment deleted from BunnyshellAvailable commands (reply to this comment):
|
❌ Preview Environment deleted from BunnyshellAvailable commands (reply to this comment):
|
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 , left 1 comment
9d1a633
to
02671fd
Compare
02671fd
to
7d1dcc7
Compare
The fix is included in the following PR: #19623 |
7d1dcc7
to
d2761a0
Compare
Signed-off-by: Keith Chong <kykchong@redhat.com>
d2761a0
to
25e753d
Compare
Signed-off-by: Keith Chong <kykchong@redhat.com>
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
/bns:deploy |
/bns:start |
/bns:start |
❌ Preview Environment deleted from BunnyshellAvailable commands (reply to this comment):
|
❌ Preview Environment deleted from BunnyshellAvailable commands (reply to this comment):
|
/bns:deploy |
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.
Thanks @saumeya @ashutosh16 for testing the PR and validating the code. I tested the PR as well and everything looks good.
Closes #18047
This change is based on top of the Edit support added from #17890
which in turn was based on the initial support that allows the viewing of multiple sources from #17275. This implements what is described in the proposal #17108.
Up to now, we have Edit support which matches what was available for single source apps (can edit and save parameter values). But this feature allows you to add and delete sources. Add was previously not available for single source apps, which would have converted a single source to a multi-source app. This feature does not allow you to do that conversion, but it allows the Add action to be performed on existing multi-source apps that use the
sources
field.The new source-panel.tsx file has some similarities with the application-create-panel.tsx. I thought it better to not change the latter and keep things separated for now, if and when someone supports creating multi-source apps from the new app panel, then it will be good to common things up.
Checklist: