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

The SDK Build Tools revision (23.0.1) is too low for project ':amazon-cognito-identity-js'. Minimum required is 25.0.0 #3

Open
mikenz1000 opened this issue Jun 5, 2018 · 0 comments

Comments

@mikenz1000
Copy link

On a clean git clone, following the instructions in README.md and then using react-native run-android to try and run on android, got this error:

FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring project ':amazon-cognito-identity-js'.
> The SDK Build Tools revision (23.0.1) is too low for project ':amazon-cognito-identity-js'. Minimum required is 25.0.0

versions:
react-native-cli: 2.0.1
react-native: 0.52.1
npm: 5.6.0
node: v8.11.1

Got past it using the android/build.gradle tweak from oblador/react-native-vector-icons#428 (comment)

But now when running the app I get this after clicking Sign Up:

Warning: Failed prop type: The prop 'onRequestClose' is marked as required in 'Modal
', but its value is 'undefined'.
in Modal (at SignUp.js:116)
in SignUp (created by Connect(SignUp))
in Connect(SignUp) (at SceneView.js:17)
...

I suspect I have a more recent SDK installed than what you had when you wrote the project, but there is an incompatibility with it? Putting a handler for onRequestClose into the Modal tag in SignUp.js prevents the error/warning (and the alert doesn't appear...)

<Modal onRequestClose={() => {
              alert('Modal has been closed.');
            }}>

but I get stuck now on the "Confirm" page where the Authentication Code input has too narrow (just shows "Au" for placeholder text), and no SMS was sent (I used all the default settings). If I need to do something else to enable SMS sending, perhaps include that in README.md, or just confirm by email for starters?

Thanks!!

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

No branches or pull requests

1 participant