Skip to content
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

nonspec: Remove outdated link to old meeting videos #1258

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

hepwori
Copy link
Contributor

@hepwori hepwori commented Dec 12, 2024

Copy link

netlify bot commented Dec 12, 2024

Deploy Preview for slsa ready!

Name Link
🔨 Latest commit fae712d
🔍 Latest deploy log https://app.netlify.com/sites/slsa/deploys/675b1c8abff498000811a403
😎 Deploy Preview https://deploy-preview-1258--slsa.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.

@hepwori hepwori changed the title Remove outdated link to old meeting videos content: Remove outdated link to old meeting videos Dec 12, 2024
@TomHennen
Copy link
Contributor

TomHennen commented Dec 13, 2024

I couldn't quite follow the thread in Slack... do we have these videos somewhere? Should we link to that?

edit: also, thanks for sending this PR!

@hepwori
Copy link
Contributor Author

hepwori commented Dec 13, 2024

I couldn't quite follow the thread in Slack... do we have these videos somewhere? Should we link to that?

They're not posted publicly any more. My thought was that we should at least remove this abandoned playlist while we figure out if we can change that.

@arewm arewm changed the title content: Remove outdated link to old meeting videos nonspec: Remove outdated link to old meeting videos Dec 13, 2024
Copy link
Member

@arewm arewm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@arewm arewm merged commit 3b4f216 into slsa-framework:main Dec 16, 2024
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants