-
Notifications
You must be signed in to change notification settings - Fork 380
Dev Center: Updates with security snippet #1078
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
I mostly don't touch legalese unless I have to. :) One edit which was originally to catch a typo, but it could use a touch of clarification. Aside from that, generally bullet points that are entire sentences should have a period at the end of them. |
Possible launch Tuesday Sept 8. |
src/partners/index.md
Outdated
5. Publish documentation. | ||
6. Submit your App for review. | ||
7. Launch into _Public Beta_! | ||
2. Review Segment's security guidance. |
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.
Hey @misteryeo - a tweak here might amp their responsibility as developers; consider changing "Review Segment's Security Guidance" to "Follow Segment's Security Guidance." That way, it more actively gets them doing what we want, as it goes beyond reading and head nodding. Well, only if it works. :)
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
…docs into security-update
This was a request from the Security and Legal teams - no ticket available as it was communicated through email. Will share with respective teams for review before merging. No urgency here but please feel free to do a review in the meantime, Laura!