-
Notifications
You must be signed in to change notification settings - Fork 28
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
Make the example app work out of the box #21
Comments
What's the contribution about? |
As I stated in the first comment, I just detailed in the README all the steps that have to be taken in order to get your example working as it doesn't work right outside the box. (maybe it used to in the past) I just wanted to know how I can open a PR so you can review it. |
The example can't work outside the box, because it needs an So, if you're willing to give away a user pool a/c for this plugin, I'd be very thankful :) |
I don't think the example works out of the box. I configured the example with my cognito configuration. My configuration is in Cognito: How do you want your end users to sign in? email -> Allow email adressed. When I enter an email as username and a password I always get: Look slike the username is not processed correctly com.amazonaws.services.cognitoidentityprovider.model.InvalidParameterException: 1 validation error detected: Value at 'username' failed to satisfy constraint: Member must satisfy regular expression pattern: [\p{L}\p{M}\p{S}\p{N}\p{P}]+ (S It looks to work when I hard code the username, e.g. return Cognito.signUo( |
I figured it out. TextController and Textbox get's often a space at the end with the user input. So a trim() will make it more stable. |
I will accept a PR that prevents typing spaces into the username field, but not one that trims user input behind the scenes. |
How can I contribute to this repo? I detailed the necessary steps to get the example working and I'd like to make a PR.
The text was updated successfully, but these errors were encountered: