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.

Sign up for GitHub

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

Jump to bottom

kFirebaseInstallationsErrorDomain missing NS_SWIFT_NAME #9275

Closed
ryanwilson opened this issue Jan 31, 2022 · 1 comment · Fixed by #9395
Closed

kFirebaseInstallationsErrorDomain missing NS_SWIFT_NAME #9275

ryanwilson opened this issue Jan 31, 2022 · 1 comment · Fixed by #9395
Labels
api: installations Swift API
Milestone
Firebase 9

Comments

@ryanwilson
Copy link
Member

firebase-ios-sdk/FirebaseInstallations/Source/Library/Public/FirebaseInstallations/FIRInstallationsErrors.h

Line 19 in bdac2b8

extern NSString *const kFirebaseInstallationsErrorDomain;

This is missing the NS_SWIFT_NAME, unfortunately fixing it is a breaking change. It should be named InstallationsErrorDomain.
@ryanwilson ryanwilson added this to the Firebase 9 milestone Jan 31, 2022
@yakovmanshin yakovmanshin mentioned this issue Feb 25, 2022
Merged
paulb777 pushed a commit that referenced this issue Mar 18, 2022
@yakovmanshin
[#9275] Added Swift Name to kFirebaseInstallationsErrorDomain (#9395)
97b93d7
@paulb777
Copy link
Member

Merged #9395 into v9
@paulb777 paulb777 mentioned this issue Mar 19, 2022
Merged
paulb777 pushed a commit that referenced this issue Mar 23, 2022
@yakovmanshin @paulb777
[#9275] Added Swift Name to kFirebaseInstallationsErrorDomain (#9395)
8568528
paulb777 pushed a commit that referenced this issue Apr 4, 2022
@yakovmanshin @paulb777
[#9275] Added Swift Name to kFirebaseInstallationsErrorDomain (#9395)
a432ac8
@firebase firebase locked and limited conversation to collaborators Apr 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: installations Swift API
2 participants
@paulb777 @ryanwilson