Closed ghost closed 1 month ago
Attention: Patch coverage is 97.69452%
with 8 lines
in your changes are missing coverage. Please review.
Project coverage is 91.00%. Comparing base (
8d5d3b2
) to head (fb1bf73
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Tested ok
Description
As we are soon going to refactor the UI of
PolicyEditor
, I have refactored the logic and components of the feature. The reason is to structure the code better for the next developer that comes in to do the issues later. Two contexts have been added:PolicyEditorContext
, which is a context for the entire policy editor.PolicyRuleContext
, which is a context for each single rule in a policy. As a result of these new contexts, a lot of prop-drilling has been removed, and components have been made smaller and more readable.Related Issue(s)
Verification