Skip to content

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