Garrett LeSage

Results 776 comments of Garrett LeSage

The cancel button here is odd. Perhaps I should add that to the hide part when bypass is active.

Force pushed with these changes: 1. Also hide the cancel button when bypassing the soft error 2. Only show separator when there's an error

I'm currently working on @allisonkarlitskaya's suggestion of the expander section. I have some ideas that are now mostly implemented. I plan on pushing up what I have soon. (Just letting...

I'm in the process of converting this to an expander. It's mostly there, but some things are broken. I'll continue tomorrow. Converted to draft to signify that this isn't done....

I don't know how we'd test this and would need assistance in that area. Can someone write tests for this please?

I still need assistance with the tests for this PR. Meanwhile: rebased.

Great to see the check improved. I think the UI as seen in this screenshot can be improved as well: ![screenshot when you're not an admin](https://user-images.githubusercontent.com/907998/192549529-e09f26bf-00f4-486f-89ce-f438fa701c90.png) Perhaps something like removing...

> We don't need to restore the old values; the idea is that React will do that on the next render. But there is a race between setting the mock...

> We could have a global toggle that puts each page into "mock mode" where it will replace certain elements with fake data. This means changes to our code, but...