-
Notifications
You must be signed in to change notification settings - Fork 409
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#10111: Enhance GeoFence attribute rule #10145
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It works as expected.
Tests performed:
- On rule creation, all the attrbutes are set to read-only, if nothing changes.
- A second edit is persisted correctly.
@tdipisa I think next steps are backport and installation in test env c040 |
@ElenaGallo @dsuren1 this is for 2024.01.01 therefore let's wait the delivery of 2024.01.00 before backporting this. Thank you. |
Why it should not be? This depends on how you decide to configure the plugin. In any case, I'm sorry but, it seems totally unrelated to the scope of this PR. Did you understand what to test here? The PR description seems quite clear to me. |
Test passed, @dsuren1 please backport to 2024.01.xx |
@dsuren1 this still need to be backported for 2024.01.01 |
Description
This PR enhances the attributes rules editor
Please check if the PR fulfills these requirements
What kind of change does this PR introduce? (check one with "x", remove the others)
Issue
What is the current behavior?
What is the new behavior?
Allows user
Breaking change
Does this PR introduce a breaking change? (check one with "x", remove the other)
Other useful information