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

LTI-257: Added shared rooms feature #260

Merged
merged 4 commits into from
Jan 4, 2024
Merged

Conversation

Mariam05
Copy link
Collaborator

Those with edit access to a room, Room X, can now enter a code from another room, Room Y, resulting in both rooms sharing the same title, description, recordings, and meeting link. Room X's moderators will not be able to edit any aspect of Room Y (ex. recordings, description, settings).

image

Clicking on the 'Use Shared Code' checkbox will allow the moderator to enter the code of another room
image

@Mariam05 Mariam05 marked this pull request as ready for review December 28, 2023 16:22
@Mariam05 Mariam05 requested a review from jfederico January 2, 2024 17:53
@jfederico jfederico changed the base branch from master to alpha January 4, 2024 18:14
@jfederico jfederico merged commit 1a1a79a into bigbluebutton:alpha Jan 4, 2024
2 checks passed
jfederico added a commit to jfederico/bbb-app-rooms that referenced this pull request Jan 23, 2024
* replace current room with shared room if use_shared_room

needs fixing:
- keep old room, only change front page (not edit)

* Added shared rooms feature

* handle user error

---------

Co-authored-by: Jesus Federico <[email protected]>
jfederico added a commit that referenced this pull request Jan 23, 2024
* LTI-257: Added shared rooms feature (#260)

* replace current room with shared room if use_shared_room

needs fixing:
- keep old room, only change front page (not edit)

* Added shared rooms feature

* handle user error

---------

Co-authored-by: Jesus Federico <[email protected]>

* LTI-257: Hide end btn in shared rooms

* LTI-257: dont allow shared room admins to edit recordings

---------

Co-authored-by: Mariam A <[email protected]>
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.

2 participants