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

Invites & sharing Room URL use Web Share API if available #6508

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

DougReeder
Copy link
Contributor

@DougReeder DougReeder commented Oct 2, 2024

Sharing is an additional option, when available. It works as expected on Firefox and Chrome on MacOS, Oculus Browser on Quest, Safari on iOS and Chrome on Android. As noted earlier, many platforms only pass the URL to destination, or just the URL and title.

Implements #6507

There's an image with this code on Docker Hub: dougreeder/hubsfirst:v1

@DougReeder
Copy link
Contributor Author

Here's a screenshot of the new Share room link using the Web Share API.

Share room link

And here's one possibility of what might result
room link email

@DougReeder
Copy link
Contributor Author

PinkPolygonSoup has said the wording is fine.

@DougReeder
Copy link
Contributor Author

That completes the changes from today's dev meeting. Now that we have testing by other people, I think this is ready to go!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant