-
Notifications
You must be signed in to change notification settings - Fork 3.7k
Improvement of Room Sharing capabilities (followup to #5068 static SIP pins) #5102
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
base: master
Are you sure you want to change the base?
Conversation
Merge Master into ics
|
Kudos, SonarCloud Quality Gate passed!
|
…into bigbluebutton-master
Update ics
|
|
We tested this and noticed that the share room link is missing the relative root path. Other than that I hope this can get merged soon, this is a very useful feature. |
|
Thanks for the feedback! I've addressed the issue. |
|
Does this PR obsolete your former one (#5068)? And do you intent to
|
|
Hi, This PR does not supersede #5068. This PR is only an update to the room sharing functionality, it works well with static PINs as well as without. I am interested in doing the rest of the work. These two PRs are my first for bigger projects so some guidance on the rubocop checks and specification would be greatly appreciated.
|
|
|
|
|












Dear reviewers,
In my previous pull request (#5068), I added support for static SIP pins. To fully leverage this feature, I've implemented a share button that opens a module allowing users to easily copy the pin, corresponding SIP phone number, and meeting link to their clipboard. Additionally, I've included an option to download an .ics file to easily send the meeting information via email.
To further enhance the user interface, I've made some restructuring changes in this pull request. These changes aim to improve the overall user experience and make the UI more intuitive and user-friendly.
I appreciate your time and feedback on this pull request.
Thank you!