Skip to content

Comments

Adding Objective-C attributes to delegate and main class to turn Objective-C runtime accessible to Nativescript#95

Merged
kuraydev merged 1 commit intokuraydev:masterfrom
dfborba:to-nativescript-projects
Oct 9, 2020
Merged

Adding Objective-C attributes to delegate and main class to turn Objective-C runtime accessible to Nativescript#95
kuraydev merged 1 commit intokuraydev:masterfrom
dfborba:to-nativescript-projects

Conversation

@dfborba
Copy link

@dfborba dfborba commented Oct 7, 2020

Hey Team, we have a project which needs to connect through WebSocket with STOMP protocol for Nativescript applications;
But to Nativescript be able to access Swift classes it needs to be able to access the Objective-C runtime. So in order to achieve it I've changed the StompClientLib and added the @objc attribute annotation to StompClientLibDelegate and @objcMembers to StompClientLib, so these classes and its attributes/methods and etc can be access from Nativescript.

Please, review this code and let me know if these changes will not affecting any other project that use this library.
Looking forward to have these changes approved 👍
Thanks.

This commit adds @objc attribute annotation to StompClientLibDelegate
and @objcMembers to StompClientLib to turn Objective-C runtime accessible
to nativescript applications
@kuraydev kuraydev self-requested a review October 7, 2020 11:39
@kuraydev kuraydev self-assigned this Oct 7, 2020
@kuraydev
Copy link
Owner

kuraydev commented Oct 7, 2020

Hello again @dfborba :)
Let me check and test it and then I will approve and release a new version :) Great job!

@kuraydev kuraydev merged commit 6835fa2 into kuraydev:master Oct 9, 2020
@kuraydev
Copy link
Owner

kuraydev commented Oct 9, 2020

Hi @dfborba,
image

I've merged and released a new version (1.4.0).

1.4.0 🥰

Thank you so much for the contribution, can you please also test the new version?

@dfborba
Copy link
Author

dfborba commented Oct 9, 2020

Amazing @WrathChaos!
Will do the tests for sure..

Happy to contribute!

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.

2 participants