-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Docs: Update RDS Identifier reference in QuickSight data source documentation #41749
base: main
Are you sure you want to change the base?
Docs: Update RDS Identifier reference in QuickSight data source documentation #41749
Conversation
Community GuidelinesThis comment is added to every new Pull Request to provide quick reference to how the Terraform AWS Provider is maintained. Please review the information below, and thank you for contributing to the community that keeps the provider thriving! 🚀 Voting for Prioritization
Pull Request Authors
|
…kSight data source documentation
b897f27
to
45653ef
Compare
@@ -264,7 +264,7 @@ To specify data source connection parameters, exactly one of the following sub-o | |||
### rds Argument Reference | |||
|
|||
* `database` - (Required) The database to which to connect. | |||
* `instance_id` - (Optional) The instance ID to which to connect. | |||
* `instance_id` - (Optional) The instance Identifier to which to connect. |
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.
Isn't "ID" already shorthand for "identifier"?
I've read the linked issue and still am unclear on the issue with the current description. Could you clarify what the this change is intended to address?
…kSight data source documentation
PR #41672: Updated documentation to reflect this addition and provide clarity on how to use the parameter when connecting to specific RDS instances.
Relations
Closes #41672
References
Output from Acceptance Testing
Not applicable, Updated the Docs.