Code review comment for ~caleb-ellis/maas:suppress-single-test

Revision history for this message
Caleb Ellis (caleb-ellis) wrote :

Lee, did you run `make force-javascript` beforehand? I've updated so only tests with status FAILED, FAILED_INSTALLING or TIMEDOUT can be suppressed via the UI. As for the table header being "Hide" I'll leave it as is for now - this was agreed upon in the design meetings and is in the 9th iteration of the wireframe (https://app.zeplin.io/project/5bfff7810f5b0001ebd72a27). If it proves to be confusing for users it's only a simple text change.

https://user-images.githubusercontent.com/25733845/56949538-1a44e800-6b2b-11e9-9dfa-6042a6def50c.png

« Back to merge proposal