Skip to content
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: do not display volume badge if it's zero #727

Merged
merged 1 commit into from
Oct 28, 2022

Conversation

benoitf
Copy link
Collaborator

@benoitf benoitf commented Oct 28, 2022

What does this PR do?

Ensure we don't display the badge for volumes if there are no volumes

Screenshot/screencast of this PR

n6dvNplqTO.mp4

What issues does this PR fix or reference?

fixes #726

How to test this PR?

Check badge is gone after deleting all volumes

Change-Id: I05627f95e075543b07b1d7cedbcb19321217d14c
Signed-off-by: Florent Benoit fbenoit@redhat.com

fixes containers#726

Change-Id: I05627f95e075543b07b1d7cedbcb19321217d14c
Signed-off-by: Florent Benoit <fbenoit@redhat.com>
@cdrage
Copy link
Contributor

cdrage commented Oct 28, 2022

Thanks for the quick fix! That was extraordinarily quick 🚀

@cdrage cdrage enabled auto-merge (squash) October 28, 2022 14:23
@cdrage cdrage merged commit 9bb6f84 into containers:main Oct 28, 2022
@podman-desktop-bot podman-desktop-bot added this to the 0.10.0 milestone Oct 28, 2022
slemeur pushed a commit to deekay2310/podman-desktop that referenced this pull request Nov 2, 2022
fixes containers#726

Change-Id: I05627f95e075543b07b1d7cedbcb19321217d14c
Signed-off-by: Florent Benoit <fbenoit@redhat.com>

Signed-off-by: Florent Benoit <fbenoit@redhat.com>
Signed-off-by: Stévan Le Meur <1636769+slemeur@users.noreply.github.com>
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.

"0" / a notification for zero volumes
3 participants