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

Draft
wants to merge 214 commits into
base: main
Choose a base branch
from
Draft

Auth swift checkpoint test #11533

wants to merge 214 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants