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

Auth swift checkpoint test #11533

Closed
wants to merge 221 commits into from
Closed

Auth swift checkpoint test #11533

wants to merge 221 commits into from

Conversation

paulb777
Copy link
Member

@paulb777 paulb777 commented Jul 12, 2023

This is a long-lived branch for reimplementing FirebaseAuth in Swift

paulb777 and others added 30 commits July 11, 2023 16:05
* Attempt at merging own ported swift code to official repo

* Fix: networkError was accidentally marked as internal.

* Added Copyright notices

* Moved base64 encoding extension. Reformatted a few todos

* Converted a bit of MultiFactor

* Added VerifyPhoneNumberRequest/Response and AuthOperationType
…10755)

* Private header exposed to internal Swift impl via private module

* Include all non-public headers in private module map

* Add TODO and #if out import

* Prefer <...> imports and fix API visibility. See more...

- The private module map enables us to selectively expose non-public
  headers to Swift. Now, the internal APIs that were made public for the
  sake of exposing to Swift can be made internal again. Swift files that
  need access to non-public API should do the following:

    @_implementationOnly import FirebaseAuth_Private

- Additionally, any public header imported into a non-public header
  needs to use <...> syntax because the header is technically coming
  from a different module (FirebaseAuth rather than FirebaseAuth_Private).

* Fix import typo

* Style

* More progress. Building for CocoaPods. Not for SPM

* Remove 'MyPrivateClass.h'

* Generate private module map

* Generate private module map followup

* Re-enable testing spec

* Style

* Remove unneeded module.modulemap

* Revert "Remove unneeded module.modulemap"

This reverts commit 3ac44c3.

* Adjust header import since FIRAuthBackend was moved

* Get library building on iOS

* Builds on iOS

* Revert private module map and update gitignore

* Add TODO

* Wrap FirebaseAuth_Private import with #if COCOAPODS

* Modularize script with arg[0] for podspec name

* Generalize script more and remove private module map

* Revert unintentionally checked in changes

* re-style @objc(...) qualifier

* Fix check

* Fix SPM build

* Change API back to what it was originally

* Fix copyright

* Better error handling in script
@paulb777
Copy link
Member Author

paulb777 commented Jul 3, 2024

Closing in favor of #13229

@paulb777 paulb777 closed this Jul 3, 2024
@firebase firebase locked and limited conversation to collaborators Aug 3, 2024
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.

5 participants