Skip to content

B1157: Multi-type registry events#5

Merged
ccomendant merged 1 commit intodevelopfrom
fix/I1456-multi-type-reg+less-like
Feb 25, 2026
Merged

B1157: Multi-type registry events#5
ccomendant merged 1 commit intodevelopfrom
fix/I1456-multi-type-reg+less-like

Conversation

@ccomendant
Copy link
Contributor

@ccomendant ccomendant commented Jan 23, 2026 •
edited
Loading

For uberAgent version 7.6 and newer:

  • update converter for B1157 to create registry rules with multiple event-types per rule.
  • partially revert 8c74551 "Disable istartswith, iendswith, icontains".

@ccomendant ccomendant requested a review from svnscha January 23, 2026 10:08
@ccomendant ccomendant force-pushed the fix/I1456-multi-type-reg+less-like branch from fc5a4d5 to ac71583 Compare February 4, 2026 23:15
svnscha
svnscha requested changes Feb 5, 2026
View reviewed changes
Copy link
Collaborator

@svnscha svnscha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes are not yet backward compatible. Please add versioning so that the updated EventType is only returned for versions >= develop. Likewise, the new like/contains behavior should only apply for versions >= develop.

@ccomendant ccomendant marked this pull request as draft February 5, 2026 13:26
@ccomendant ccomendant force-pushed the fix/I1456-multi-type-reg+less-like branch from ac71583 to eca41bf Compare February 5, 2026 14:20
@ccomendant ccomendant marked this pull request as ready for review February 5, 2026 14:21
@ccomendant ccomendant marked this pull request as draft February 5, 2026 14:29
@ccomendant
Copy link
Contributor Author

ccomendant commented Feb 5, 2026

@svnscha, please merge your PR#6 into main so that I can rebase this PR on top of it.

@ccomendant ccomendant force-pushed the fix/I1456-multi-type-reg+less-like branch from eca41bf to 41292d0 Compare February 24, 2026 14:16
@ccomendant ccomendant changed the base branch from main to develop February 24, 2026 14:18
@ccomendant ccomendant marked this pull request as ready for review February 24, 2026 14:18
@ccomendant ccomendant force-pushed the fix/I1456-multi-type-reg+less-like branch from 41292d0 to 36e2c89 Compare February 24, 2026 14:27
@ccomendant ccomendant requested a review from svnscha February 25, 2026 08:30
@svnscha svnscha closed this Feb 25, 2026
@svnscha svnscha deleted the fix/I1456-multi-type-reg+less-like branch February 25, 2026 08:42
@svnscha svnscha restored the fix/I1456-multi-type-reg+less-like branch February 25, 2026 08:43
@svnscha svnscha reopened this Feb 25, 2026
svnscha
svnscha approved these changes Feb 25, 2026
View reviewed changes
Copy link
Collaborator

@svnscha svnscha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice

@ccomendant ccomendant merged commit 205d5a0 into develop Feb 25, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@svnscha svnscha svnscha approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ccomendant @svnscha