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

Hub callback bug fixes & invokeMethod improvements #12

Merged
merged 12 commits into from
Jan 20, 2021
Merged

Conversation

AyonAB
Copy link
Contributor

@AyonAB AyonAB commented Nov 1, 2020

Changes made in this PR:

  • Fix a issue where hub callback only accepting strings.

  • Hub callback now returns the message as well as the subscribed method name.

  • Make invokeMethod generic.

  • As many arguments as you want in invokeMethod.

  • Update documentation.

Note: This PR should be merged after the new version is pushed to the Pub.

- invokeMethod now supports upto 10 arguments
- statusChangeCallback type changes to dynamic (Possible fix for a reported issue)
- Hub method callback type changed to Any (Android)
- Android gradle updates

Signed-off-by: Ayon Das <ayantorres@gmail.com>
Signed-off-by: Ayon Das <ayantorres@gmail.com>
…method name

(iOS testing WIP)

Signed-off-by: Ayon Das <ayantorres@gmail.com>
@AyonAB AyonAB added bug Something isn't working enhancement New feature or request labels Nov 1, 2020
@AyonAB AyonAB self-assigned this Nov 1, 2020
@AyonAB AyonAB changed the title Hub callback bug fixes Hub callback bug fixes & invokeMethod improvements Nov 1, 2020
- HubCallback now returns the methodName and Result
- invokeMethod now can take any number of arguments
- Few other minor fixes

Signed-off-by: Ayon Das <ayantorres@gmail.com>
Signed-off-by: Ayon Das <ayantorres@gmail.com>
Signed-off-by: Ayon Das <ayantorres@gmail.com>
- subscribeToHubMethod will be dropped
- Hub methods list added, this will be the new way to subscribe to hubs

Signed-off-by: Ayon Das <ayantorres@gmail.com>
- Update documentation

Signed-off-by: Ayon Das <ayantorres@gmail.com>
Signed-off-by: Ayon Das <ayantorres@gmail.com>
Signed-off-by: Ayon Das <ayantorres@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

java.lang.IllegalStateException: Expected STRING but was BEGIN_OBJECT at path
1 participant