-
Notifications
You must be signed in to change notification settings - Fork 4
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
aws-cognito and other changes #13
Conversation
Aws cognito
tryRecoverUser is now recoverUser. Closes #2
README.md
Outdated
</aws-app> | ||
<aws-dynamodb | ||
table-name="notes" | ||
data="{{data}}"> | ||
table-name="notes" |
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 think we should use 4 indentations instead of 5
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.
yeah this is a typo
README.md
Outdated
</aws-app> | ||
<aws-lambda | ||
id="transformNotes" |
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.
Same as previous comment but the other way around
aws-cognito.html
Outdated
}, | ||
|
||
/** | ||
* Tries to recover a user session from localStorage. If user is recovered |
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.
Once space to much for indentation
See updates from f718673 |
This PR adds an aws-cognito element as well as a few new features for aws-dynamodb. It also introduces and complies with linting rules and updates documentation.