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

Document how to join a Jitsi Meeting #103

Merged
merged 3 commits into from
Jan 27, 2021
Merged

Document how to join a Jitsi Meeting #103

merged 3 commits into from
Jan 27, 2021

Conversation

Cathryne
Copy link
Contributor

Shorter alternative to #48. This here is written for users who receive a meeting link created by someone else, thus skips the Open meet… URL step from PR #48.

By linking to the browser's own docu about the permissions popups, we can also ensure that no docu here needs to be kept up to date with browser UI updates.

Copy link
Contributor

@toby63 toby63 left a comment

Choose a reason for hiding this comment

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

Looks like a good start 🙂 .
Some comments.

docs/user-guide/join-a-jitsi-meeting.md Outdated Show resolved Hide resolved
docs/user-guide/join-a-jitsi-meeting.md Outdated Show resolved Hide resolved
docs/user-guide/join-a-jitsi-meeting.md Outdated Show resolved Hide resolved
@Cathryne
Copy link
Contributor Author

Cathryne commented Jan 4, 2021

Looks like a good start 🙂 .
Some comments.

Thanks! Implemented. Have a good start into the new year :-)

Copy link
Contributor

@toby63 toby63 left a comment

Choose a reason for hiding this comment

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

New Review.

Sry 😅.

@@ -4,11 +4,17 @@ title: Join a Jitsi Meeting
sidebar_label: Join a Jitsi Meeting
---

# Join by using a Jitsi link
Copy link
Contributor

@toby63 toby63 Jan 8, 2021

Choose a reason for hiding this comment

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

I would just write link or weblink.
More correctly would probably be url or web address, but I think link is better understandable for most people worldwide.

Jitsi link might be a bit missunderstandable.

Please refert to the browser's documentation for details (e.g.
People invite each other to Jitsi meetings mostly by sending a link.

1. If you have received such an invite link from a trusted source,
Copy link
Contributor

Choose a reason for hiding this comment

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

I would replace trusted source with a person you trust.

Copy link
Contributor

Choose a reason for hiding this comment

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

🤔 Hmm, now that I think about it, it might be good to mention both a person you trust and trust in the server/serverowner.

This way we could leave out Trust in Server below 👇 .

1. If you have received such an invite link from a trusted source,
copy it into your browser and press <kbd>Enter</kbd>/<kbd>Return</kbd>.
2. Your browser may first ask you to grant microphone and/or camera access.
If you trust the person who invited you, confirm this access request.
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 tricky 🤔 ...but I would add some more information like:

  1. Microphone and Camera are not strictly necessary (Only allow, if you want others to see and hear you) and can be used seperately (e.g. microphone only).
  2. Trust in server
  3. Trust in other people in the room

Also maybe use the word allow (instead of confirm), as these are the terms used at least in firefox (if i remember correctly).
I think it might be easier for people to understand.

@saghul saghul merged commit 69dd058 into jitsi:master Jan 27, 2021
@saghul
Copy link
Member

saghul commented Jan 27, 2021

I'm landing this as is, it's a good start!

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.

3 participants