forked from btafel/cordova-plugin-braintree
-
Notifications
You must be signed in to change notification settings - Fork 42
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Include missing frameworks in ios tag in plugin.xml
- Loading branch information
1 parent
008ceb4
commit b5bf8cb
Showing
1 changed file
with
14 additions
and
11 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
b5bf8cb
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Seems obvious that this should/would fix the issues I ran into w/ the
BraintreePaymentFlow
reference inBraintreeDropIn
b5bf8cb
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@adamfellon Yep! There's one other issue, which I'm not sure the most straightforward fix; if you are using PayPal with the plugin, you need to hard code your
BUNDLE_ID.payments
string in the URL schemes in*-Info.plist
. (e.g. if your app iscom.myapp.excellent
, hardcodecom.myapp.excellent.payments
into the URL schemes instead of the${PRODUCT_BUNDLE_IDENTIFIER}.payments
scheme)This is because cordova still uses the REALLY old version of that variable, so the script that Taraque included doesn't work (not this plugin's fault - cordova needs to modernize the template).
So just make sure you do that manually when building.