feat(auth, ios): add fullName support for Sign in with Apple credentials#9096
feat(auth, ios): add fullName support for Sign in with Apple credentials#9096russellwheatley wants to merge 6 commits into
Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request introduces support for capturing and forwarding the user's full name during the initial Sign in with Apple flow on iOS. By extending the credential creation process and updating the native bridge, the library now allows developers to pass name components that Firebase uses to populate the user's display name. Note that this feature is iOS-specific, as the Firebase Android SDK does not currently support credential-level name injection; on Android, developers should continue to use Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize the Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counterproductive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds support for forwarding the user's full name (fullName) during Sign in with Apple on iOS, allowing Firebase to store it as the account's display name on first sign-in. This is implemented by updating the TypeScript types, the native bridge on iOS to parse fullName into NSPersonNameComponents, and adding corresponding unit and type tests. Android and Web platforms gracefully ignore this field to maintain spec parity. The review feedback suggests two improvements in RNFBAuthModule.mm: optimizing the Apple provider check to avoid unnecessary dictionary parsing and allocation, and ensuring personNameComponentsFromDictionary: returns nil if no valid name components are present to prevent passing empty objects to the Firebase SDK.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #9096 +/- ##
============================================
+ Coverage 58.61% 58.73% +0.12%
Complexity 1616 1616
============================================
Files 503 503
Lines 39178 39210 +32
Branches 5807 5810 +3
============================================
+ Hits 22962 23027 +65
+ Misses 14811 14780 -31
+ Partials 1405 1403 -2
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
I'm enjoying watching these creep up, even if ever so slowly, now that coverage is on everywhere and e2e is pretty reliably working+uploading the reports for tracking: 
|
2 participants
Description
Adds support for forwarding Apple's
fullName(NSPersonNameComponents) to Firebase when signing in with theapple.comprovider, so the user's display name can be set on first authorization instead of being lost.AppleFullPersonNametype and an optionalfullNamefield onOAuthCredentialOptions.AppleAuthProvider.credential()andOAuthProvider('apple.com').credential()now accept and forwardfullNameon the resultingOAuthCredential(including throughtoJSON/fromJSON).signInWithCredentialforwardsfullNametoFIROAuthProvider.appleCredentialWithIDToken:rawNonce:fullName:, so Firebase stores it asdisplayNameon the newly created account.fullNameis accepted (for TurboModule spec parity) and intentionally ignored; apps should callupdateProfile()after sign-in on Android.firebase-js-sdk'sOAuthCredentialhas nofullNamefield on any platform); the parameter is accepted and ignored.signInWithCredentialsignature.Related issues
Linear: CP-158 (Auth: support Apple fullName in Sign in with Apple credential).
closes #8570
Release Summary
Sign in with Apple credentials now support forwarding the user's
fullName, so it can be stored asdisplayNameon first sign-in (iOS only; ignored on Android/Web).Checklist
AndroidiOSOther(macOS, web)e2etests added or updated inpackages/\*\*/e2ejesttests added or updated inpackages/\*\*/__tests__Test Plan
yarn tests:jest packages/auth— 147/147 tests passing, including new coverage forfullNamemapping, JSON round-tripping, and native bridge forwarding.yarn lint:js,yarn tsc:compile,yarn compare:types— all clean.yarn lint:ios:check(clang-format) andgoogle-java-format— native changes conform.node ./scripts/codegen-verify.mjs— regenerated codegen artifacts match exactly, no drift in any other package.Think
react-native-firebaseis great? Please consider supporting the project with any of the below:React Native FirebaseandInvertaseon Twitter