-
Notifications
You must be signed in to change notification settings - Fork 161
Design QA Instructions
Shixie edited this page Jun 8, 2021
·
33 revisions
- 1. Review PR's changes against Definition of done in Storybook preview links
- 2. Review Percy results
- 3. Comment on PR to request for changes
- 4. Repeat above until no changes are needed
- 5. Approve PR and remove
Needs design reviewlabel.
Review PR's changes against Definition of done in Storybook preview links. Keep in mind there might be multiple storybooks to review. In general,
- a style package change affects both Web components and React storybooks.
- a new component build in Web components should be reviewed in React wrapper storybook, and not React storybook.
- an experimental component change will require the
Feature flaglabel to generate the Experimental components storybook preview link. - a RTL (Right to Left language) change will require
RTLlabel to generate the RTL components storybook preview link

- Review Percy snapshots: there should be no unexpected diffs
- Comment to request changes
Points to keep in mind:
- Be conscientious to the PR's specific goal. If you notice a new issue unrelated (and not triggered by) the current issue, create a new issue.