Skip to content

feat: add password reset for default admin/user accounts#1463

Closed
jescalada wants to merge 6 commits intofinos:mainfrom
jescalada:1022-force-admin-password-reset-fixed
Closed

feat: add password reset for default admin/user accounts#1463
jescalada wants to merge 6 commits intofinos:mainfrom
jescalada:1022-force-admin-password-reset-fixed

Conversation

@jescalada
Copy link
Copy Markdown
Contributor

@jescalada jescalada commented Mar 20, 2026

Draft PR for #1022, attempting to debug failing E2E tests from #1457

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 20, 2026

Deploy Preview for endearing-brigadeiros-63f9d0 canceled.

Name Link
🔨 Latest commit 6d974de
🔍 Latest deploy log https://app.netlify.com/projects/endearing-brigadeiros-63f9d0/deploys/69bce9fb5aa3950008f6c1a5

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 20, 2026

Codecov Report

❌ Patch coverage is 76.10063% with 38 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.22%. Comparing base (758b2db) to head (6d974de).

Files with missing lines Patch % Lines
src/service/routes/auth.ts 69.38% 29 Missing and 1 partial ⚠️
src/service/passport/passwordChangeHandler.ts 44.44% 5 Missing ⚠️
src/service/routes/utils.ts 62.50% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1463      +/-   ##
==========================================
- Coverage   89.66%   89.22%   -0.45%     
==========================================
  Files          68       69       +1     
  Lines        4869     5020     +151     
  Branches      888      921      +33     
==========================================
+ Hits         4366     4479     +113     
- Misses        485      522      +37     
- Partials       18       19       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jescalada
Copy link
Copy Markdown
Contributor Author

jescalada commented Mar 20, 2026

Closing, decided not to implement regeneration/logout. Original PR is ready for review #1457

@jescalada jescalada closed this Mar 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant