Vulnerability scanner: Add ability to see all items on a server if one is an administrator#322
Merged
Vulnerability scanner: Add ability to see all items on a server if one is an administrator#322
Conversation
dotNomad
approved these changes
Feb 9, 2026
Collaborator
dotNomad
left a comment
dotNomad
left a comment
There was a problem hiding this comment.
Let's add an item to the CHANGELOG for this too 🎉
Noted a couple things we could improve, but nothing necessary. Awesome change
Comment on lines
+215
to
+227
| <label v-if="userStore.isAdmin" class="flex items-center gap-2 cursor-pointer"> | ||
| <span class="text-sm text-gray-600">show:</span> | ||
| <span class="text-sm" :class="showAllContent ? 'text-gray-600' : 'text-gray-800 font-medium'">only mine</span> | ||
| <div class="relative"> | ||
| <input | ||
| type="checkbox" | ||
| v-model="showAllContent" | ||
| class="sr-only peer" | ||
| /> | ||
| <div class="w-11 h-6 bg-gray-200 peer-focus:outline-none peer-focus:ring-4 peer-focus:ring-blue-300 rounded-full peer peer-checked:after:translate-x-full rtl:peer-checked:after:-translate-x-full peer-checked:after:border-white after:content-[''] after:absolute after:top-[2px] after:start-[2px] after:bg-white after:border-gray-300 after:border after:rounded-full after:h-5 after:w-5 after:transition-all peer-checked:bg-blue-600"></div> | ||
| </div> | ||
| <span class="text-sm" :class="showAllContent ? 'text-gray-800 font-medium' : 'text-gray-600'">all</span> | ||
| </label> |
Collaborator
There was a problem hiding this comment.
Would be nice, but not necessary to break this into a separate component.
Comment on lines
+29
to
+31
| async function fetchContentList(forceRefresh: boolean = false) { | ||
| // Skip if content is already loaded for this mode (unless forcing refresh) | ||
| if (!forceRefresh && isContentLoaded.value && contentList.value.length > 0) return; |
Collaborator
There was a problem hiding this comment.
Using this it was a bit jarring to flip back and forth and have the entire UI reload. I wonder if we could store the content separately so once loaded a use can swap back and forth.
Definitely too big for this PR though since that would require a substantial refactor.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Checks if someone is an administrator, and if they are adds a toggle to see all packages.
resolves: #321