-
Notifications
You must be signed in to change notification settings - Fork 277
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: Allow to install in one click featured extensions (if not installed) #2355
Conversation
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.
II see this error when fetching the catalog Unable to fetch the available extensions: Timeout awaiting 'lookup' for 1000ms
and it results in the N/A status. Not sure what we can do to improve it but if that happens, what's the point of showing the crc/openshift local extension if i cannot do anything?
Other point is that the icon of the extension are not high quality - look at the podman/lima/local icons. We should update them
podman-desktop/packages/main/src/plugin/extensions-catalog/extensions-catalog.ts Lines 95 to 102 in 4e90e5b
|
yes, by increasing it it worked fine. |
need to find the good limit then |
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.
Approving it as the code related to my issues is at #2346 and i'll be out for a while.
If we can enhance the icons and increase the timeout or make it customizable would be great.
#2346 has been merged now, so we will need to rebase |
@cdrage rebased |
…lled) related to containers#1901 widget available from dashboard page and extensions list Change-Id: I495ad06dd7f27a3779195c3fc503dae3dd6e0cc3 Signed-off-by: Florent Benoit <fbenoit@redhat.com> Change-Id: I1559329c48dd4d9390e133d8ddcf647340dded92
I'll open follow-up PRs for timeouts and icons |
Requires:
What does this PR do?
Display a widget in dashboard and extensions list to install featured extension in one click
Screenshot/screencast of this PR
62PolDwZOo.mp4
What issues does this PR fix or reference?
related to #1901
How to test this PR?
It requires commits from
There are component tests
How to test: Launch Podman Desktop (and check that for example you don't already have SandBox or OpenShift Local being installed)
Then click on the download icon and wait a little, then extension should be there
Change-Id: I495ad06dd7f27a3779195c3fc503dae3dd6e0cc3