cncf / tag-security

πŸ”CNCF Security Technical Advisory Group -- secure access, policy control, privacy, auditing, explainability and more!
https://cncf.io/projects
Other
1.99k stars 496 forks source link

Remove SCS from navbar, add Publications #1265

Closed eddie-knight closed 3 weeks ago

eddie-knight commented 3 weeks ago

Part of #1257

Preview

netlify[bot] commented 3 weeks ago

Deploy Preview for tag-security ready!

Name Link
Latest commit b337d4ede798d8f03ecd152efad52e3ce3813623
Latest deploy log https://app.netlify.com/sites/tag-security/deploys/666206b1094c560008fd4f1a
Deploy Preview https://deploy-preview-1265--tag-security.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

eddie-knight commented 3 weeks ago

Navbar and the contents of the publications page look to be resolved - but any link to markdown file (directly in the repo) is having the .md stripped and resulting in a 404.

Got itβ€” this is clashing with another patch that we put in to remove the .md from local links. I'll investigate.

mrcdb commented 3 weeks ago

Related to this, I just noticed PUBLICATIONS.md does not include the links to all the latest CNS WP v2 translations (e.g. Italian): https://deploy-preview-1265--tag-security.netlify.app/publications/#cloud-native-security-whitepaper

Updating the PUBLICATIONS.md file with the new links may be an additional action item in https://github.com/cncf/tag-security/issues/1257 once this is merged.

eddie-knight commented 3 weeks ago

Good shout @mrcdb. @anvega, can you confirm that you'll be able to help update the publications file? Should we track that as a new gh issue?

eddie-knight commented 3 weeks ago

Had to revise the link handling, but it should all be working now!