-
Notifications
You must be signed in to change notification settings - Fork 120
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
[Feature] - Add Submit API #682
Conversation
dtroupe-plaid
commented
Jun 13, 2024
•
edited
Loading
edited
- Update Android SDK to 4.5.0.
- Update iOS SDK to 5.6.0.
- Add submit API.
@@ -63,7 +63,7 @@ android { | |||
compileSdkVersion 31 | |||
|
|||
defaultConfig { | |||
minSdkVersion rootProject.ext.safeExtGet("minSdkVersion", 21) | |||
minSdkVersion rootProject.ext.has("minSdkVersion") ? rootProject.ext.minSdkVersion : 21 |
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.
This was changed in this PR, but it was causing compile errors for me so I modified it.
@@ -97,6 +100,18 @@ export function PlaidLinkScreen() { | |||
}}> | |||
<Text style={styles.button}>Open Link</Text> | |||
</TouchableOpacity> | |||
<TouchableOpacity |
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.
I left this here so testing would be easier. I'll remove after review.
@melissaosullivan @wkiefer there are some changes in this PR to make testing easier (package.lock etc). I will clean that up after review 😄 You should be able to test doing the following: cd example
npm run start
npm run android cd example/ios
bundle install
pod install
cd ../
npm run start
npm run ios |