Skip to content

Adding Community MRs review

Nadia Udalova requested to merge nudalova-master-patch-86948 into master

Why is this change being made?

The goal of this MR is to help ensuring a better review of the community contributions that involve UX and/or UI changes. It is caused by Adding more guidance around UX and Engineering collaboration discussion.

Currently, Product Designers sometimes are not being aware of the changes UX and/or UI that are caused by the community contributions.

Proposal

Add instructions on UX and Engineering alignment around Community MRs review and approval.

Ideas are based on the conversations with Product Designers and Engineering team - more ideas and feedback are welcome!

Author Checklist

  • Provided a concise title for the MR
  • Added a description to this MR explaining the reasons for the proposed change, per [say-why-not-just-what][transparency]
  • Assign this change to the correct DRI
    • If the DRI for the page/s being updated isn’t immediately clear, then assign it to your manager.
    • If your manager does not have merge rights, please ask someone to merge it AFTER it has been approved by your manager in [#mr-buddies][mr-buddies-slack].
    • If the changes relate to any part of the project other than updates to content and/or data files please make sure to ping @gl-static-site-editor in a comment for a review and merge. For example changes to .gitlab-ci.yml, JavaScript/CSS/Ruby code or the layout files.

Merge request reports