-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Update the list of domains that should be whitelisted #690
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
✅ Validation status: passed
For more details, please refer to the build report. Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report. |
@keithrob This is in tech review; FYI. |
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 don't believe the installer uses these at all; are you thinking of the download of the actual bootstrapper (note: the section you're adding to is specific to the installation, the later section is around usage in the product)?
✅ Validation status: passed
For more details, please refer to the build report. Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report. |
✅ Validation status: passed
For more details, please refer to the build report. Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report. |
@TerryGLee & @ikeras , |
@ikeras Thanks, Anson. |
#sign-off |
Update the list of domainsthat should be whitelisted.