Skip to content

userId != userID#8

Merged
treeder merged 1 commit into
treeder:mainfrom
iced:main
Mar 27, 2026
Merged

userId != userID#8
treeder merged 1 commit into
treeder:mainfrom
iced:main

Conversation

@iced

@iced iced commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates the property name from userId to userID within the passkey options in src/passkeys.js. A review comment identifies a potential runtime error where sess.userId could be undefined if a session is not found, and suggests implementing an authentication check to ensure the user is logged in before proceeding.

Comment thread src/passkeys.js
@treeder
treeder merged commit ca60c60 into treeder:main Mar 27, 2026
2 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.

2 participants