-
Notifications
You must be signed in to change notification settings - Fork 284
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
fix: updates extension state if installed/uninstalled from CLI #5060
fix: updates extension state if installed/uninstalled from CLI #5060
Conversation
99d3cf1
to
5a1629e
Compare
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.
This change appears to introduce some unwanted side effects in determining which extensions are installed via the UI
2023-06-27.07-38-45.mp4
@rak-phillip I think it's ready for another quick pick, I did some refactor moving the part where we check for the state (installed/uninstalled) to the catalog. |
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.
This change is looking great, but there's a few things we can clean up before moving forward.
@@ -46,7 +46,7 @@ import { Banner } from '@rancher/components'; | |||
|
|||
import LoadingIndicator from '@pkg/components/LoadingIndicator.vue'; | |||
import demoMetadata from '@pkg/utils/_demo_metadata.js'; | |||
import { ipcRenderer } from '@pkg/utils/ipcRenderer'; | |||
// import { ipcRenderer } from '@pkg/utils/ipcRenderer'; |
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.
We can remove commented imports
We merged #5073 to add spell check to CI, so a rebase on top of main might be in order |
Signed-off-by: scures <scurescu@suse.com>
Signed-off-by: scures <scurescu@suse.com>
Signed-off-by: scures <scurescu@suse.com>
Signed-off-by: scures <scurescu@suse.com>
Signed-off-by: scures <scurescu@suse.com>
bacefb6
to
bd2eaad
Compare
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.
One more change and we should be good to merge 🙂
Signed-off-by: scures <scurescu@suse.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
Fixes #4951
Updates the extension state using
ipcRenderer