-
Notifications
You must be signed in to change notification settings - Fork 249
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
Add documentation on public test suites #2572
Add documentation on public test suites #2572
Conversation
Preview URL: https://2572--bk-docs-preview.netlify.app |
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 reckon it could be worth expanding on how to turn on public suites, as only org admins will have this power initially, until the org TA settings are updated to grant this permission to all users. Maybe something like:
"Permission to make a test suite public is reserved for organization admins initially. Admins are able to extend this permission to all organization members via the Security tab in the organization settings."
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.
LGTM; would be good to get a review from @mbelton-buildkite before merging
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've left a bunch of suggestions, but in general, it looks good to me. 😊
b7646f9
to
1038c47
Compare
NOTE: This should not be merged until the feature flag
ANALYTICS_PUBLIC_SUITE_ACCESS
is turned on for all usersWe are about to release a new feature that introduces public test suites into Test Analytics. We need to make sure users understand what public suites are so it is used appropriately. There is a risk that users will make their suite public without realising the privacy implications if it is not documented.
I have copied what is existing in the docs for Public pipelines and altered the content to relate to test suites
Linear ticket