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

Added NS_ERROR_ENUM to Firebase Auth, breaking changes (#7723) #9000

Merged
merged 2 commits into from
Mar 31, 2022

Conversation

a-25
Copy link
Contributor

@a-25 a-25 commented Nov 23, 2021

Possible fix for issue (#7723) FR: Support NS_ERROR_ENUM for better error handling in Swift
Breaking changes

Note from @ryanwilson: this is a one line change with 347 lines of formatting changes.

@google-cla google-cla bot added the cla: yes label Nov 23, 2021
@a-25 a-25 marked this pull request as draft November 23, 2021 17:37
@paulb777 paulb777 added this to the Firebase 9 milestone Nov 24, 2021
@a-25 a-25 changed the title [WIP] Added NS_ERROR_ENUM to Firebase Auth, breaking changes (#7723) Added NS_ERROR_ENUM to Firebase Auth, breaking changes (#7723) Dec 6, 2021
@a-25 a-25 marked this pull request as ready for review January 29, 2022 10:08
@ryanwilson
Copy link
Member

Thanks for the contribution - this is now going through API review. #9512 covers the rest of the API surface to move at the moment.

@ryanwilson
Copy link
Member

cc @rosalyntan for the code that's associated with the API review.

@ryanwilson ryanwilson changed the base branch from master to v9b March 31, 2022 21:39
@ryanwilson
Copy link
Member

Changed the base to the V9 branch where this will be released - merging now. Thanks for the contribution!

@ryanwilson ryanwilson merged commit 7db1ebc into firebase:v9b Mar 31, 2022
paulb777 pushed a commit that referenced this pull request Apr 4, 2022
* Added NS_ERROR_ENUM to Firebase Auth, breaking changes (#7723)

* Added NS_ERROR_ENUM to Firebase Auth, breaking changes (#7723), code style
@firebase firebase locked and limited conversation to collaborators May 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants