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

POJO enhancements #233

Merged
merged 14 commits into from Jul 5, 2017
Merged

POJO enhancements #233

merged 14 commits into from Jul 5, 2017

Conversation

om26er
Copy link
Contributor

@om26er om26er commented Jul 4, 2017

  • Add multiple Session.publish() implementations
  • Add back convenience Session.call() aka max-comfort.

@oberstet We can add more convenience methods to Session.call() as well like you can see for Session.publish(), are you fine with more convenience methods ?

Also it is still not clear how the signature of a method that sends a list of POJOs would look like.

@om26er
Copy link
Contributor Author

om26er commented Jul 4, 2017

This commit: ce64ca6 introduces two new subscribe methods. First takes a method with single parameter as argument and the second takes two parameters as arguments. Need opinion if we should got with that approach.

@om26er
Copy link
Contributor Author

om26er commented Jul 4, 2017

Added some new APIs for Session.subscribe() and Session.register() <-- POJO variants.

@om26er
Copy link
Contributor Author

om26er commented Jul 5, 2017

I pushed more APIs though this branch may not really be fully working yet as there are POJO scenarios that still need to be developed.

@oberstet oberstet merged commit d58ae62 into crossbario:master Jul 5, 2017
@om26er om26er deleted the pojo-publish branch July 20, 2017 11:08
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.

None yet

2 participants