Greg Anderson

Results 36 comments of Greg Anderson

@coheigea planning to merge after 2.15 release on tuesday.

@X0x1RG9f happy to merge with unit tests.

@pna-nca I believe are helm people are away right now :( I apologize this can't be reviewed sooner.

@vankyver will merge after the release.

@blakeaowens no need to test, if the JS change looks safe to you, please just let me know.

Reviewing this as part of our new weekly PR review. We like the idea of this PR, but there are some issues two quick examples: setting debugging to true and...

Closing due to lack or response. Please feel free to re-engage when/if you are able.

I see Stefan's point, but I would assume most users would expect the copy to not modify data, rather than us trying to modify it smartly.

@blakeaowens is going to try to fix the zindex issue.

This is a cool PR, but not in-line with DefectDojo functionality, modifying the engagement, etc.