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

Update blogs' frontmatter URI to match the directory structure #360

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ddelnano
Copy link
Member

Through the process of debugging why Hackernews posts are linking to the wrong post, I realized that the "eBPF TLS tracing: The Past, Present and Future" and "eBPF Probes and You: Navigating the kernel source for tracing" posts are one of the few posts that have their frontmatter inconsistent with the directory structure.

Our code seems to use the directory path as the URI (loading the blog posts confirms this), so the frontmatter is not used. While I don't have proof this is causing the strange Hackernews behavior, I do know that my "Keeping prod observable" post was posted properly. The pages I fixed in this PR had the incorrect meta tag issue as seen in the screenshots below:

hackernews

Copy link

netlify bot commented Sep 16, 2024

Deploy Preview for blog-px-dev ready!

Name Link
🔨 Latest commit b26d970
🔍 Latest deploy log https://app.netlify.com/sites/blog-px-dev/deploys/66e8bca1a9cc8600083117a9
😎 Deploy Preview https://deploy-preview-360--blog-px-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 82 (🟢 up 4 from production)
Accessibility: 71 (no change from production)
Best Practices: 100 (no change from production)
SEO: 76 (no change from production)
PWA: 80 (no change from production)
View the detailed breakdown and full score reports

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

@ddelnano ddelnano changed the title Update blog's frontmatter URI to match the directory structure Update blogs' frontmatter URI to match the directory structure Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants