Skip to content

Conversation

@ivanpauno
Copy link
Collaborator

@ivanpauno ivanpauno commented Aug 20, 2020

This PR implements event handlers for subscriptions.

I haven't added bindings for a subscription status yet, so I have not written tests.
Next PR is going to be that. After that, we can merge feature/events branch into foxy-devel.

Signed-off-by: Ivan Santiago Paunovic <[email protected]>
@ivanpauno ivanpauno added the enhancement New feature or request label Aug 20, 2020
@ivanpauno ivanpauno requested a review from jacobperron August 20, 2020 16:47
@ivanpauno ivanpauno self-assigned this Aug 20, 2020
Copy link

@jacobperron jacobperron left a comment

Choose a reason for hiding this comment

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

Minor comments. LGTM

ivanpauno and others added 2 commits August 24, 2020 13:34
Signed-off-by: Ivan Santiago Paunovic <[email protected]>

Co-authored-by: Jacob Perron <[email protected]>
Signed-off-by: Ivan Santiago Paunovic <[email protected]>

Co-authored-by: Jacob Perron <[email protected]>
@ivanpauno ivanpauno merged commit 4fd64b3 into feature/events Aug 24, 2020
@ivanpauno ivanpauno deleted the ivanpauno/support-events-in-subscriptions branch August 24, 2020 16:56
ivanpauno added a commit that referenced this pull request Aug 31, 2020
ivanpauno added a commit that referenced this pull request May 17, 2021
ivanpauno added a commit to ros2-java/ros2_java that referenced this pull request Jan 14, 2022
ivanpauno added a commit to ros2-java/ros2_java that referenced this pull request Jan 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants