Revise line to emphasize screenshots
What does this MR do?
Revises the comment after the 'Screenshots' subheading in the default merge request template to make clearer that screenshots are required for UI changes.
Related issues
Related to Update the Screenshots section of the default M... (#324356 - closed).
Author's checklist
-
Optional. Consider taking the GitLab Technical Writing Fundamentals course. -
Follow the: -
If you're adding or changing the main heading of the page (H1), ensure that the product tier badge is added. -
If you are a GitLab team member, request a review based on: - The documentation page's metadata.
- The associated Technical Writer.
If you are a GitLab team member and only adding documentation, do not add any of the following labels:
~"frontend"
~"backend"
~"type::bug"
~"database"
These labels cause the MR to be added to code verification QA issues.
Reviewer's checklist
Documentation-related MRs should be reviewed by a Technical Writer for a non-blocking review, based on Documentation Guidelines and the Style Guide.
-
If the content requires it, ensure the information is reviewed by a subject matter expert. - Technical writer review items:
-
Ensure docs metadata is present and up-to-date. -
Ensure the appropriate labels are added to this MR. -
Ensure a release milestone is set. - If relevant to this MR, ensure content topic type principles are in use, including:
-
The headings should be something you'd do a Google search for. Instead of Default behavior
, say something likeDefault behavior when you close an issue
. -
The headings (other than the page title) should be active. Instead of Configuring GDK
, say something likeConfigure GDK
. -
Any task steps should be written as a numbered list. - If the content still needs to be edited for topic types, you can create a follow-up issue with the docs-technical-debt label.
-
-
-
Review by assigned maintainer, who can always request/require the reviews above. Maintainer's review can occur before or after a technical writer review.
Merge request reports
Activity
changed milestone to %15.3
assigned to @aqualls
Suggested Reviewers (beta)
The individuals below may be good candidates to participate in the review based on various factors.
You can use slash commands in comments to quickly assign
/assign_reviewer @user1
.Suggested Reviewers @eread
,@godfat-gitlab
,@kwiebers
,@markglenfletcher
,@caalberts
If you do not believe these suggestions are useful, please apply the label Bad Suggested Reviewer. You can also provide feedback for this feature on this issue:
https://gitlab.com/gitlab-org/gitlab/-/issues/357923
.Automatically generated by Suggested Reviewers Bot - an experimental ML-based recommendation engine created by ~"group::applied ml".
- Resolved by Suzanne Selhorn
@jiaan @uhlexsis I'm finally getting around to this one-line change now that I'm back from vacation!
You're both marked Busy right now, so I'm copying you as a courtesy. I'll route this change over to @annabeldunstone for a first review, and then to a technical writer. Sorry about the vacation-based delay!
requested review from @annabeldunstone
1 Warning Please add a merge request subtype to this merge request. If needed, you can retry the
danger-review
job that generated this comment.Generated by
Dangerremoved review request for @annabeldunstone
requested review from @sselhorn
requested review from @andr3, @csouthard, and @crystalpoole and removed review request for @sselhorn
requested review from @sselhorn
enabled an automatic merge when the pipeline for b84360b9 succeeds
mentioned in commit 39205bbb
added workflowstaging-canary label
added workflowcanary label and removed workflowstaging-canary label
added workflowstaging label and removed workflowcanary label
added workflowproduction label and removed workflowstaging label
added workflowpost-deploy-db-staging label and removed workflowproduction label
added workflowpost-deploy-db-production label and removed workflowpost-deploy-db-staging label
added maintenancerefactor label
added releasedcandidate label
added releasedpublished label and removed releasedcandidate label