Skip to content

Conversation

MojtabaHs
Copy link
Contributor

Fix a typo in a static constant name. I couldn't see any use of this static property. Please double check and let me know if we should update elsewhere related to this property as well.
Thanks

@ncooke3
Copy link
Member

ncooke3 commented Feb 27, 2025

Thanks, @MojtabaHs. I think since there is no usage of this key, it coule be removed entirely.

@MojtabaHs MojtabaHs changed the title fix: user defaults key for remove user property time in seconds chore: remove unused user defaults key for remove user property time in seconds property Feb 27, 2025
Copy link
Contributor

@andrewheard andrewheard left a comment

Choose a reason for hiding this comment

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

Thanks for catching an unused constant!

@ncooke3 ncooke3 merged commit 1d32f36 into firebase:main Feb 28, 2025
28 checks passed
@MojtabaHs MojtabaHs deleted the FirebaseIAM branch March 10, 2025 09:01
@firebase firebase locked and limited conversation to collaborators Mar 31, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants