-
-
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
fix: portfolio example JSX error #3886
Conversation
🦋 Changeset detectedLatest commit: 2833719 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
@@ -20,7 +20,7 @@ const projects = (await Astro.glob("./project/**/*.md")) | |||
s | |||
most | |||
recent | |||
projects' | |||
projects |
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.
Ah this must have happened in #3862!
Would you mind updating the description
attribute so that it says:
description="Learn about Jeanine White's most recent projects"
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.
Yes, introduced here: https://github.com/withastro/astro/pull/3862/files#diff-a739195a828a765d1e4de3a8e8bf4622ea9b80dc79ac379432843fcc5447a946R23
I've updated the description!
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.
Oops, totally missed this in my review of that PR, my apologies!
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.
Looks great, thank you!
Changes
I created a new Astro project with the
portfolio
template and noticed a JSX error insrc/pages/projects.astro
(an extra'
after theprojects
prop), so this PR remove it.I've also noticed that the dev server starts and render the page without any error/bug with this extra
'
, so this is not really a bug, more of a typo.Testing
No test was added since this concern the portfolio example.
Docs
No doc was changed since this concern the portfolio example.