Closed Bug 1629712 Opened 9 months ago Closed 5 months ago

Update Code Reviews Checklist

Categories

(DevTools :: Documentation, enhancement, P3)

enhancement

Tracking

(firefox81 fixed)

RESOLVED FIXED
81 Branch
Tracking Status
firefox81 --- fixed

People

(Reporter: victoria, Assigned: Honza)

Details

Attachments

(1 file)

Looks like due to the conversion to Sphinx, the Code Reviews Checklist bulleted lists became flattened by a level (compare to the old version).

Also, I'd like to update this sentence:

If a new piece of UI or new user interaction is added/modified, then UX is ui-review? on the bug.

To this:

If any user-facing interfaces are added/modified, double-check the changes with the UX mockups or specs, if available. If there's any confusion, need-info the UX designer.

I can see the issue, thanks for the report Victoria.

Honza

Severity: -- → S3
Priority: -- → P3
Assignee: nobody → odvarko
Status: NEW → ASSIGNED
Pushed by jodvarko@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5fe2c56a6dcf
Fix documentation text. r=victoria
Status: ASSIGNED → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → 81 Branch
You need to log in before you can comment on or make changes to this bug.