Follow-up from "Conditionally render policy alert editor"
The following discussions from !52036 (merged) should be addressed:
-
@kerrizor started a discussion: Nit: I wonder if we could extract the large, repetitive common elements here into a
shared_example
or even alet
variable?🤔 -
@kerrizor started a discussion: Nit: I might consider pulling this into a
private
method (#details
even) just to clean up the readability of this method, but that's super minor.