-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
docs(misc): adding nx live episode w Mike and some fixes #27837
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
1f53449
to
287fee4
Compare
☁️ Nx Cloud ReportCI is running/has finished running commands for commit 8e6152c. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this CI Pipeline Execution ✅ Successfully ran 4 targets
Sent with 💌 from NxCloud. |
@@ -3,8 +3,9 @@ title: Better security with Personal Access Tokens | |||
slug: personal-access-tokens | |||
authors: ['Philip Fulcher'] | |||
tags: [nx-cloud] | |||
cover_image: /blog/images/2024-09-06/personal-access-tokens-header.png | |||
cover_image: /blog/images/2024-09-10/personal-access-tokens-header.png |
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.
This should use personal-access-tokens-header.avif
can we add it back?
The .png
file should also be there so that it can be used as the file for OG image.
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.
cool added it back. btw @ndcunningham - it looked like there were some type errors around the usage of the ogImage
properties on master? it was in blog/[slug]/page.tsx
that I saw it
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.
Isn't the file has been deleted can you restore the file?
959d209
to
a242a95
Compare
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.
LGTM
This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request. |
Current Behavior
Expected Behavior
Related Issue(s)
Fixes #