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

Add DynamoDBReconfigurePolicy #615

Merged
merged 3 commits into from
Oct 10, 2018
Merged

Conversation

JohanZackrisson
Copy link
Contributor

Add DynamoDBReconfigurePolicy which grants the user access to dynamo db:UpdateTable
https://docs.aws.amazon.com/amazondynamodb/latest/APIReference/API_UpdateTable.html

Issue #, if available:
#614

Description of changes:
Added DynamoDBReconfigurePolicy, which grants dynamodb:Update

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Copy link
Contributor

@keetonian keetonian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the contribution! Could you add this policy into the tests as in https://github.com/awslabs/serverless-application-model/pull/589/files? And thank you for providing a link to the documentation, we'll review this policy template submission soon!

@keetonian
Copy link
Contributor

Also, if you haven't heard of Hacktoberfest, go ahead and take a look! It's a collaboration between a few companies to promote open source contributions during the month of October, and this pull request will count for it.

@JohanZackrisson
Copy link
Contributor Author

Added the testcases.
Thanks for pointing me to the other pull request. 🥇

@keetonian
Copy link
Contributor

Thank you for updating your PR and for providing proper resource scoping in your Policy Template!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants