-
Notifications
You must be signed in to change notification settings - Fork 13
docs: Add public accessibility requirement for Redshift destination #1303
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
base: main
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| You need admin privileges in Amplitude, and a role that allows you to enable resources in Redshift. | ||
|
|
||
| {{partial:admonition type="warning" heading="Redshift endpoint must be publicly accessible"}} | ||
| Your Redshift endpoint must be publicly accessible for Amplitude to connect. Private endpoints accessible only within your VPC aren't supported. |
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.
Passive voice used in warning admonition (Bugbot Rules)
The phrase "aren't supported" uses passive voice, which violates Rule #1 (Active Voice - Critical Priority). The style guide requires 100% active voice and flags constructions like "is/are [verb]ed". Consider rephrasing to active voice, such as "Amplitude doesn't support private endpoints accessible only within your VPC."
|
I've added new changes to address this Slack mention in commit 8d00eb9 |
|
I've added new changes to address this Slack mention in commit 800a11e |
|
Just a reminder: If you'd like me to act on any feedback you have via Github comments, just type @Promptless in your suggestion and I'll get right on it! (I won't show up in the user dropdown, but I'll process any request that has @Promptless in the comment body.) |
Adds a warning admonition in the Prerequisites section to explicitly state that Redshift endpoints must be publicly accessible. Addresses connection failures caused by private-only endpoints accessible only within VPCs.
Trigger Events
Open this suggestion in the Promptless dashboard
Help us improve Promptless — If this suggestion missed the mark, please share quick feedback.
If you want Promptless to make further changes on this PR, feel free to leave a comment tagging Promptless (It won't show up in the user drop down but Promptless will get it!)
Note
Docs update
content/collections/destination-catalog/en/amazon-redshift.mdPrerequisites, adds explicit note that Amplitude supports only public Redshift endpoints and private endpoints aren't supported.Written by Cursor Bugbot for commit 800a11e. This will update automatically on new commits. Configure here.