Swift wrapper #45
Swift wrapper #45
Conversation
| #import <Razorpay/Razorpay.h> | ||
| #import <Razorpay/RazorpayPaymentCompletionProtocolWithData.h> |
There was a problem hiding this comment.
@AbhinavArora95 This is the change in the react native wrapper that I was talking about. If people try to run the new framework with the wrapper on npm right now they will get a compile error because of missing ExternalWalletSelectionProtocol and RazorpayPaymentCompletionProtocolWithData Let's push an update to npm once you are back.
Also I think we can make the integration easier for merchants. What are you thoughts on these:
-
Let's make our documentation clearer explaining why we are not shipping the framework with the wrapper and redirecting them to the correct page for downloading the different versions.
-
The package.json for the example app should point to npm and not the local copy as it does right now given that the idea of this is to show the merchants how their package.json should look like and not development of the wrapper itself.
-
All the additional steps that we have mentioned other than maybe adding the framework to the project can be converted to a single script. Let's do that and the readme can just show how to execute the script.
-
Some of the additional steps like making the changes to the AppDelegate file seems to be not needed anymore. I was able to run my sample project without those changes. Can we remove those?
No description provided.