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

Update README.md for visionOS #12293

Merged
merged 3 commits into from
Jan 17, 2024
Merged

Update README.md for visionOS #12293

merged 3 commits into from
Jan 17, 2024

Conversation

paulb777
Copy link
Member

#no-changelog

See also internal cl/599193113 for updates to the platform support table matrix

Copy link
Contributor

@andrewheard andrewheard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Paul, this and the corresponding CL LGTM!

Copy link
Contributor

@morganchen12 morganchen12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: backticks around FIREBASE_SOURCE_FIRESTORE

@paulb777 paulb777 merged commit fc08f4d into main Jan 17, 2024
37 checks passed
@paulb777 paulb777 deleted the paulb777-patch-1 branch January 17, 2024 17:29
@@ -237,14 +237,25 @@ To receive push notifications, follow the steps above and run the app on a physi

## Building with Firebase on Apple platforms

Firebase 8.9.0 introduced official beta support for macOS, Catalyst, and tvOS. watchOS continues
to be community supported. Thanks to community contributions for many of the multi-platform PRs.
Firebase 8.9.0 introduced official beta support for macOS, Catalyst, and tvOS. visionOS and watchOS
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a bit confusing.

The first part of the sentence mentions Firebase 8.9.0, which adds macOS, Catalyst, and tvOS. It goes on to talk about visionOS.

I think we should write "In Firebase 10.xx.x, we added support for visionOS. visionOS and watchOS are community-supported...

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@firebase firebase locked and limited conversation to collaborators Feb 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants