-
Notifications
You must be signed in to change notification settings - Fork 125
tvOS Reference Docs Changes #1175
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved with a few typo nits, please fix
auth/src/include/firebase/auth.h
Outdated
@@ -174,7 +174,7 @@ class Auth { | |||
void set_language_code(const char* language_code); | |||
|
|||
/// Sets the user-facing language code to be the default app language. This | |||
/// uses a languge associated with the phone's locale data. On desktop | |||
/// uses a languge associated with the device's locale data. On desktop |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While you're here, can you fix the typo "languge" => "language"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@@ -168,7 +168,7 @@ class Settings final { | |||
* `set_dispatch_queue()`. | |||
* | |||
* @note This method is only available when `__OBJC__` is defined, such as | |||
* when compiling for iOS. | |||
* when compiling for iOS or tvOS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing period
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
@@ -170,7 +170,7 @@ struct Notification { | |||
|
|||
/// Indicates the key to the title string for localization. | |||
/// | |||
/// On iOS, this corresponds to "title-loc-key" in APNS payload. | |||
/// On iOS amd tvOS, this corresponds to "title-loc-key" in APNS payload. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Integration test with FLAKINESS (succeeded after retry)Requested by @DellaBitta on commit b1a5444
Add flaky tests to go/fpl-cpp-flake-tracker |
Update the reference docs to refer to tvOS when discussing particular methods and classes.
Description
Update the reference docs to refer to tvOS when discussing particular methods and classes.
Testing
Packaging Build for docs push
Type of Change
Place an
x
the applicable box: