Draft: Protected packages + containers: Add count icon to list in settings UI
What does this MR do and why?
- Adding count icon (package icon + number of entries) to
the tables
protected packages
andprotected containers
- This icon element was suggested by @annabeldunstone in !146523 (comment 1838085406)
- This MR addresses on point in the implementation plan, see #413641
Changelog: added
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
MR Checklist (@gerardo-navarro)
-
Changelog entry added, if necessary -
Documentation created/updated via this MR -
Documentation reviewed by technical writer or follow-up review issue created -
Tests added for this feature/bug -
Tested in all supported browsers -
Conforms to the code review guidelines -
Conforms to the merge request performance guidelines -
Conforms to the style guides -
Conforms to the javascript style guides -
Conforms to the database guides
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
Before | After |
---|---|
How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.
Related to #413641
Merge request reports
Activity
added typefeature label
added Leading Organization label
Hey @gerardo-navarro!
Thank you for your contribution to GitLab. Please refer to the contribution documentation for an overview of the process.
When you're ready for a first review, post
@gitlab-bot ready
. If you know a relevant reviewer(s) (for example, someone that was involved in a related issue), you can also assign them directly with@gitlab-bot ready @user1 @user2
.At any time, if you need help, feel free to post
@gitlab-bot help
or initiate a mentor session on Discord. Read more on how to get help.You can comment
@gitlab-bot label <label1> <label2>
to add labels to your MR. Please see the list of allowed labels in thelabel
command documentation.This message was generated automatically. You're welcome to improve it.
added Community contribution workflowin dev labels
assigned to @gerardo-navarro
2 Warnings This merge request does not refer to an existing milestone. Labels missing: please ask a reviewer or maintainer to add backend, frontend, documentation to this merge request. 1 Message This merge request adds or changes documentation files. A review from the Technical Writing team before you merge is recommended. Reviews can happen after you merge. Documentation review
The following files require a review from a technical writer:
-
doc/api/graphql/reference/index.md
(Link to current live version)
The review does not need to block merging this merge request. See the:
-
Metadata for the
*.md
files that you've changed. The first few lines of each*.md
file identify the stage and group most closely associated with your docs change. - The Technical Writer assigned for that stage and group.
- Documentation workflows for information on when to assign a merge request for review.
Multiversion compatibility
This merge request updates GraphQL backend and frontend code.
To prevent an incident, ensure the updated frontend code is backwards compatible.
For more information, see the multiversion compatibility documentation.
Reviewer roulette
Category Reviewer Maintainer backend @praba.m7n
(UTC+2)
@hustewart
(UTC-4)
frontend @sheldonled
(UTC+1)
@arfedoro
(UTC+2)
Please check reviewer's status!
Please refer to documentation page for guidance on how you can benefit from the Reviewer Roulette, or use the GitLab Review Workload Dashboard to find other available reviewers.
If needed, you can retry the
danger-review
job that generated this comment.Generated by
Danger-
added linked-issue label
mentioned in issue #413641
mentioned in issue #441345 (closed)
added 1 commit
- b4819a81 - Protected packages + containers: Add count icon to list in settings UI
added 1 commit
- b4819a81 - Protected packages + containers: Add count icon to list in settings UI
mentioned in merge request !146523 (merged)
mentioned in issue gitlab-org/quality/triage-reports#17094 (closed)
mentioned in issue gitlab-org/quality/triage-reports#17178 (closed)
Hey @gerardo-navarro is this MR ready for review ?
added devopspackage grouppackage registry sectionci labels
@gerardo-navarro, it seems we're waiting on an action from you for approximately two weeks.
- Do you still have capacity to work on this? If not, you might want to close this MR and/or ask someone to take over.
- Do you need help in getting it ready? At any time, you can:
- If you're actually ready for a review, you can post
@gitlab-bot ready
.
This message was generated automatically. You're welcome to improve it.
added automation:author-reminded label
Security policy violations have been resolved.
Edited by GitLab Security Botmentioned in issue #480012