Skip to content

Fix missing comma in session config example - #14647

Merged
ArmandPhilippot merged 2 commits into
withastro:mainfrom
nasty23-star:fix/session-config-comma
Oct 1, 2026
Merged

ArmandPhilippot merged 2 commits into
withastro:mainfrom
nasty23-star:fix/session-config-comma

Conversation

@nasty23-star

Copy link
Copy Markdown
Contributor

Description (required)

https://docs.astro.build/en/guides/sessions/

Problem
In the "Configuring sessions" example, adapter: vercel() is missing a trailing comma before session: {, which makes the snippet invalid JS.

Proposed fix
Add a comma after vercel().

References

@astrobot-houston

astrobot-houston commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the tracking.ignoredKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
guides/sessions.mdx Source changed, localizations will be marked as outdated.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@github-actions

github-actions Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Preview deployment

✅ Deployment complete!

@ArmandPhilippot ArmandPhilippot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice catch, thank you! And welcome to Docs contributors! 🥳

@ArmandPhilippot
ArmandPhilippot merged commit 9be0a62 into withastro:main Oct 1, 2026
9 checks passed
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.

Missing comma in session config example on /en/guides/sessions/

3 participants