Skip to content
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

ACS-5325 Invoke required policy #2215

Merged
merged 21 commits into from
Oct 3, 2023

Conversation

damianujma
Copy link
Contributor

This PR is to enable sending events for the PROP_CASCADE_TX property change and the parent timestamp propagation.

@dsibilio dsibilio self-requested a review September 28, 2023 09:18
Copy link
Contributor

@dsibilio dsibilio left a comment

Choose a reason for hiding this comment

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

Please update copyright headers for the necessary classes.

@damianujma damianujma requested a review from dsibilio September 29, 2023 13:18
Copy link
Contributor

@dsibilio dsibilio left a comment

Choose a reason for hiding this comment

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

LGTM, left a minor suggestion due to the aspect/property inconsistency previously pointed out.

Copy link
Contributor

@jamalkm jamalkm left a comment

Choose a reason for hiding this comment

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

LGTM.

@damianujma damianujma merged commit 456adc2 into master Oct 3, 2023
@damianujma damianujma deleted the feature/ACS-5325_invoke_required_policy branch October 3, 2023 11:28
Sbisht19 pushed a commit that referenced this pull request Oct 4, 2023
---------

Co-authored-by: Domenico Sibilio <[email protected]>
HylandAditya pushed a commit that referenced this pull request Oct 12, 2023
---------

Co-authored-by: Domenico Sibilio <[email protected]>
code4uuuu pushed a commit that referenced this pull request Oct 12, 2023
---------

Co-authored-by: Domenico Sibilio <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants