-
Notifications
You must be signed in to change notification settings - Fork 3.7k
enh: Add contextual popovers for recording visibility help #6136
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
enh: Add contextual popovers for recording visibility help #6136
Conversation
|
@Ndaboom I suggest the inline documentation follow the same order. Ex.: Public/Protected is first in my instance. |
677a164 to
b5c44af
Compare
b5c44af to
1589025
Compare
Done, Thank you. |
|
I hope @ffdixon and the team will like this :-) |
|
Thanks for submitting the PR - this is a great addition. Maybe just some small changes regarding the wording used: Public/Protected - The recording is available on the Public Recordings page that can be reached from the Room Join page. The link can only be viewed from within Greenlight |
|
Requested changes to the wording have now been applied. Please let me know if anything else needs an update. |
|
Related: #6064 |



Related issue: #5816