Skip to content

Conversation

@JerryVincent
Copy link
Collaborator

@JerryVincent JerryVincent commented Feb 4, 2026

Type of Change

  • Dependency upgrade
  • Bug fix (non-breaking change)
  • Breaking change
    • e.g. a fixed bug or new feature that may break something else
  • New feature
  • Code quality improvements
    • e.g. refactoring, documentation, tests, tooling, ...

Implementation

The users can now see the status of their email confirmation on the app\routes\settings.account.tsx route. If their email is not confirmed, an "Email not Confirmed" badge will be shown under the email input field. Otherwise it shows "Email Confirmed". If the email is not confirmed, a button appears that can be used for resending the confirmation email, and this calls the api "app\routes\api.users.me.resend-email-confirmation.ts". Included the translations as well. The re-confirmation email is sent successfully if the user clicks the resend confirmation email button, as I have tested it with MailHog.

Checklist

  • I gave this pull request a meaningful title
  • My pull request is targeting the dev branch
  • I have added documentation to my code
  • I have deleted code that I have commented out

Additional Information

@scheidtdav
Copy link
Member

scheidtdav commented Feb 9, 2026

@JerryVincent Is there a corresponding issue for this? If so, please add it.

…not and request to resend the confirmation mail if they are not confirmed.
…r web UI and falls back to JWT auth for API clients/tests.
…ts\routes\api.users.me.boxes.$deviceId.spec.ts.
…terall hooks in the file tests\routes\api.users.me.boxes.$deviceId.spec.ts
@JerryVincent JerryVincent force-pushed the feat/showEmailconfirmationStatus branch from 78b181a to 08daacd Compare February 11, 2026 10:27
@scheidtdav scheidtdav enabled auto-merge (squash) February 11, 2026 12:58
@scheidtdav scheidtdav merged commit 2e11c92 into dev Feb 11, 2026
12 of 14 checks passed
@scheidtdav scheidtdav deleted the feat/showEmailconfirmationStatus branch February 11, 2026 13:50
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.

handle user accounts with unverified email addresses properly

2 participants