-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
r/aws_sns_sms_preferences Fix documentation omission #4694
Conversation
2e19cec
to
91a51e9
Compare
91a51e9
to
8127e86
Compare
8127e86
to
2453822
Compare
2453822
to
74e8623
Compare
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.
Thanks for catching this! 🚀 It will be released with v1.21.0 of the AWS provider later this week.
@@ -19,6 +19,7 @@ BUG FIXES: | |||
* resource/aws_budgets_budget: Force new resource when updating `name` [GH-4656] | |||
* resource/aws_dms_endpoint: Additionally specify MongoDB connection info in the top-level API namespace to prevent issues connecting [GH-4636] | |||
* resource/aws_rds_cluster: Prevent additional retry error during S3 import for IAM/S3 eventual consistency [GH-4683] | |||
* resource/aws_sns_sms_preferences: Properly add SNS preferences to website docs [GH-4694] |
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.
FYI the maintainers usually manage the CHANGELOG and we generally do not denote documentation updates (as there are usually many more of these per release)
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.
Solid - my bad I just did it out of habit. Feel free to change it if you like!
This has been released in version 1.21.0 of the AWS provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks! |
r/sns_sms_peferences
is missing from the documentation and also the filename was spelled incorrectly.