Skip to content
Snippets Groups Projects

Added warning about quick actions

Merged Marcel Kornegoor requested to merge kornegoor/gitlab:master-patch-6898 into master
All threads resolved!

What does this MR do?

Improves quality and completeness of documentation of issue CSV import functionality

Related issues

#389924 (closed)

Author's checklist

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 you aren't sure which tech writer to ask, use roulette or ask in the #docs Slack channel.

  • 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 like Default behavior when you close an issue.
      • The headings (other than the page title) should be active. Instead of Configuring GDK, say something like Configure 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

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Evan Read changed milestone to %15.9

    changed milestone to %15.9

  • Evan Read requested review from @georgekoltsov and removed review request for @eread

    requested review from @georgekoltsov and removed review request for @eread

  • 2 Warnings
    :warning: 4da7a197: The commit subject must start with a capital letter. For more information, take a look at our Commit message guidelines.
    :warning: 0a5a6a58: The commit subject may not be longer than 72 characters. For more information, take a look at our Commit message guidelines.
    1 Message
    :book: 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:

    The review does not need to block merging this merge request. See the:

    If needed, you can retry the :repeat: danger-review job that generated this comment.

    Generated by :no_entry_sign: Danger

  • George Koltsov approved this merge request

    approved this merge request

  • George Koltsov requested review from @eread and removed review request for @georgekoltsov

    requested review from @eread and removed review request for @georgekoltsov

  • added 1 commit

    Compare with previous version

  • Evan Read changed title from Added warning about quick actions (labels, milestones and users must exist) to Added warning about quick actions

    changed title from Added warning about quick actions (labels, milestones and users must exist) to Added warning about quick actions

  • Evan Read approved this merge request

    approved this merge request

  • Evan Read requested review from @wortschi and removed review request for @eread

    requested review from @wortschi and removed review request for @eread

  • @wortschi, this Community contribution was recently assigned to you for review.

    • Do you still have capacity to review this? We are mindful of your time, so if you are not able to take this on, please re-assign to one or more other reviewers.
    • Add the workflowin dev label if the merge request needs action from the author.

    This message was generated automatically. You're welcome to improve it.

  • Evan Read
  • Evan Read added 1 commit

    added 1 commit

    • 6d117150 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Evan Read resolved all threads

    resolved all threads

  • Evan Read enabled an automatic merge when the pipeline for b6f5d7c6 succeeds

    enabled an automatic merge when the pipeline for b6f5d7c6 succeeds

  • Evan Read requested review from @eread

    requested review from @eread

  • merged

  • @kornegoor, how was your code review experience with this merge request? Please tell us how we can continue to iterate and improve:

    1. React with a :thumbsup: or a :thumbsdown: on this comment to describe your experience.
    2. Create a new comment starting with @gitlab-bot feedback below, and leave any additional feedback you have for us in the comment.

    Interested in learning more tips and tricks to solve your next challenge faster? Subscribe to the GitLab Community Newsletter for contributor-focused content and opportunities to level up.

    Thanks for your help! :heart:

    This message was generated automatically. You're welcome to improve it.

  • Evan Read mentioned in commit 9c7de6a4

    mentioned in commit 9c7de6a4

  • added workflowstaging label and removed workflowcanary label

  • 🤖 GitLab Bot 🤖 added groupimport and integrate label and removed 1 deleted label

    added groupimport and integrate label and removed 1 deleted label

  • Please register or sign in to reply
    Loading