Conversation
✅ Deploy Preview for cockroachdb-interactivetutorials-docs canceled.
|
✅ Deploy Preview for cockroachdb-api-docs canceled.
|
Files changed:
|
✅ Netlify Preview
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Netlify Preview
To edit notification comments on pull requests, go to your Netlify project configuration. |
taroface
left a comment
taroface
left a comment
There was a problem hiding this comment.
A couple minor suggestions. However, I'm not sure I agree with
Move "Technical advisories" down to reference section which is more appropriate than releases
The TAs are release-specific contingencies that someone contemplating an upgrade may want to see. They also don't feel consistent with general "Reference" material. The way that they're presented feels reminiscent of a changelog, since this is a running list of notices. I suggest leaving this under Releases for now.
@taroface I think the context in which someone wants to know about a technical advisory is when the advisory is posted, rather than at a time when they're considering upgrading more broadly, which is why I don't generally think of TAs as release material. My proposal here is that it's a low-risk opportunity to further simplify the releases nav, which is already dense with the variety of release notes captured. Case in point, Starburst docs bundle technical advisories in with other high-level security information despite having advisories associated with affected and mitigated product versions. Let me know what you think, that's my argument but if you disagree I'm comfortable reverting this one. |
Co-authored-by: Ryan Kuo <8740013+taroface@users.noreply.github.com>
2 participants
https://cockroachlabs.atlassian.net/browse/DOC-16117