Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#678 fixed removing email from userRecord via update_user #744

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

smithlauren910
Copy link

Discussion

This PR resolves issue #678.

Testing

Suggestions are welcomed for further testing.

@Adeline09
Copy link

Description of changes:
In firebase_admin/_user_mgt.py we changed the email validation in the payload to include the condition that checks whether email should be deleted via DELETE_ATTRIBUTE. Also we wrote the function to delete email from the database upon checking DELETE_ATTRIBUTE.

Description of tests:
In tests/test_user_mgt.py we included corresponding fields for email in the function test_update_user_delete_fields. Then we wrote the function test_update_user_delete_provider_and_email that verifies email deletion from the database in the same manner as phone number deletion.

@lahirumaramba lahirumaramba self-assigned this Dec 19, 2023
@prameshj
Copy link

cc @renkelvin

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.

4 participants