Skip to content

email magic code autocomplete#2938

Merged
huumn merged 2 commits intostackernews:masterfrom
Soxasora:fix/autofill-magic-code
Apr 27, 2026
Merged

email magic code autocomplete#2938
huumn merged 2 commits intostackernews:masterfrom
Soxasora:fix/autofill-magic-code

Conversation

@Soxasora
Copy link
Copy Markdown
Member

@Soxasora Soxasora commented Apr 27, 2026

Description

This PR adds support for magic code autocomplete in the email form via autocomplete='one-time-code', browsers (and email clients/operating systems) that support this will retrieve the magic code from the email we send.

Screenshots

autocomplete-otp.mov

Additional Context

  • check if other forms need autocomplete support

Checklist

Are your changes backward compatible? Please answer below:

Yes

On a scale of 1-10 how well and how have you QA'd this change and any features it might affect? Please answer below:

9, minimal change


Note

Low Risk
Low risk UI-only change that adds an autocomplete hint to the email magic-code input; no auth logic or data handling is modified.

Overview
Adds autoComplete='one-time-code' to the magic-code MultiInput on pages/email.js so supporting browsers/OSes can autofill the emailed verification code.

Reviewed by Cursor Bugbot for commit 82fd2dd. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 0d85646. Configure here.

Comment thread pages/email.js Outdated
Comment thread pages/email.js Outdated
@huumn
Copy link
Copy Markdown
Member

huumn commented Apr 27, 2026

cool!

@huumn huumn merged commit e929d3f into stackernews:master Apr 27, 2026
7 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