Skip to content

fix(api): handle users without email addresses#60467

Merged
ShaunSHamilton merged 1 commit intofreeCodeCamp:mainfrom
ojeytonwilliams:fix/handle-null-emails
May 22, 2025
Merged

fix(api): handle users without email addresses#60467
ShaunSHamilton merged 1 commit intofreeCodeCamp:mainfrom
ojeytonwilliams:fix/handle-null-emails

Conversation

@ojeytonwilliams
Copy link
Copy Markdown
Contributor

Checklist:

This isn't the whole solution, but it should prevent the errors while we come up with a way to let affected users know.

Ref: #60465

@ojeytonwilliams ojeytonwilliams requested a review from a team May 21, 2025 15:22
@github-actions github-actions bot added the platform: api Server application that needs familiarity with Express, Fastify, MongoDB etc. label May 21, 2025
@sentry
Copy link
Copy Markdown

sentry bot commented May 21, 2025

Sentry Issue: API-FASTIFY-D

@moT01 moT01 added the MERGE CONFLICT! To be applied to PR's that have a merge conflict and need updating label May 21, 2025
@ojeytonwilliams ojeytonwilliams force-pushed the fix/handle-null-emails branch from 1c4ed67 to 260c718 Compare May 22, 2025 08:40
@ojeytonwilliams ojeytonwilliams added status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending. and removed MERGE CONFLICT! To be applied to PR's that have a merge conflict and need updating labels May 22, 2025
Copy link
Copy Markdown
Member

@ShaunSHamilton ShaunSHamilton left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@ShaunSHamilton ShaunSHamilton merged commit 848ae3a into freeCodeCamp:main May 22, 2025
17 checks passed
@ojeytonwilliams ojeytonwilliams deleted the fix/handle-null-emails branch May 22, 2025 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

platform: api Server application that needs familiarity with Express, Fastify, MongoDB etc. status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants