Skip to content

Only spins for UX reviewer if that team includes a designer

Marcel van Remmerden requested to merge mvanremmerden-master-patch-71352 into master

What does this MR do and why?

We only want to recommend a UX review for teams that include a Product Designer: gitlab-org/gitlab!120763 (merged)

The goal is:

graph TD
    A[If MR has UX label] --> B[Is it a community contribution?]
    B -->|Yes| C[Does that team have a designer?]
    C -->|Yes| D[Suggest that teams designer as reviewer]
    C -->|No| E[Suggest Pedro]
    B -->|No| F[Does that team have a designer?]
    F -->|Yes| G[Spin for random designer]
    F -->|No| H[Don't suggest any designer]

Demo at: gitlab-org/gitlab!123816 (merged)

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Lin Jen-Shin

Merge request reports