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

Update README.md #58

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

abbasidaniyal
Copy link

This is an important point that is missing. Took me an hour to figure it out

This is an important point that is missing. Took me an hour to figure it out
@jakejscott
Copy link

@abbasidaniyal I think this has been addressed by another author here: #63

You should leave 'generate client secret' checked for native apps. It's just when your using javascript sdk you leave it unchecked https://github.com/aws-amplify/amplify-js/tree/master/packages/amazon-cognito-identity-js#configuration

@doc-rj-celltrak
Copy link

The readme update would have helped me too, and it applies until PR #63 is merged. Unfortunately, this project seems to be orphaned at the moment.

@furaiev
Copy link

furaiev commented Dec 23, 2019

Hi all here,
I've copied this project in a separate package (because this one isn't supported anymore) https://pub.dev/packages/amazon_cognito_identity_dart_2
Welcome 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.

4 participants