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

Improve DittoProtocol MessagePath to be aware of message subject #1641

Merged
merged 1 commit into from Jun 20, 2023

Conversation

thjaeckle
Copy link
Member

  • this was very cumbersome to determine before when using the MessagePath API
  • also added boolean isInboxOutboxMessage() for determining if the message path represents an inbox/outbox message

* this was very cumbersome to determine before when using the `MessagePath` API
* also added `boolean isInboxOutboxMessage()` for determining if the message path represents an inbox/outbox message

Signed-off-by: Thomas Jäckle <thomas.jaeckle@beyonnex.io>
@thjaeckle thjaeckle added this to the 3.3.0 milestone May 24, 2023
@thjaeckle thjaeckle self-assigned this May 24, 2023
@thjaeckle thjaeckle changed the title improve DittoProtocol MessagePath to be aware of message subject Improve DittoProtocol MessagePath to be aware of message subject May 24, 2023
@thjaeckle
Copy link
Member Author

@alstanchev @kalinkostashki I would appreciate a review here as well :)

Copy link
Contributor

@kalinkostashki kalinkostashki left a comment

Choose a reason for hiding this comment

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

LGTM :)

@thjaeckle thjaeckle merged commit 9dd1f78 into master Jun 20, 2023
3 checks passed
@thjaeckle thjaeckle deleted the feature/message-path-api-improvements branch June 20, 2023 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants