-
Notifications
You must be signed in to change notification settings - Fork 372
IP Allowlisting GA [DOC-986] #6892
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
Conversation
✅ Deploy Preview for segment-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
🔥 looks great, just one suggestion
> Segment might add additional IP address ranges. Before adding additional ranges, Segment will send an email to all Workspace Owners notifying them of the update. | ||
|
||
### Supported destinations | ||
Segment supports IP Allowlisting in all Classic and Actions destinations except for the following: |
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.
Classic and Actions destinations as these are internal names for categorization, so customers likely may not understand this.
Couple of suggestions to resolve (pls feel free to ignore if there are more elegant solutions)
- reword to All destinations in Catalog (link to the Catalog) except the following
- add links to Classic and Actions destinations
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'll go with the first option! Good callout, thanks!
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.
Looks good except one minor comment
✅ Deploy Preview for segment-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site. |
Proposed changes
Added section to Destinations Overview page + notes on some pages about new IP Allowlisting functionality
Merge timing
On/before 8/7
Related issues (optional)