Skip to content

Add documentation for BBB.#81

Merged
CeEv merged 2 commits intomainfrom
add_bbb
Jan 15, 2026
Merged

Add documentation for BBB.#81
CeEv merged 2 commits intomainfrom
add_bbb

Conversation

@CeEv
Copy link
Contributor

@CeEv CeEv commented Jan 15, 2026

Description

Links to Tickets or other pull requests

Screenshots of UI changes

Approval for review

  • DEV: If the API or client code was changed, all necessary code generation or synchronization steps were completed and tested.
  • QA: In addition to review, the code has been manually tested (if manual testing is possible)
  • All points were discussed with the ticket creator, support-team or product owner. The code upholds all quality guidelines from the PR-template.

@CeEv CeEv self-assigned this Jan 15, 2026
@CeEv CeEv enabled auto-merge (squash) January 15, 2026 12:02
Add the BBB credentials **HOST** and **SALT** to your env. file, you can find them in 1Password.

Add `FEATURE_VIDEOCONFERENCE_ENABLED=true` in **client** and **server** and
`FEATURE_VIDEOCONFERENCE_WAITING_ROOM_ENABLED=true` to your env. file in the client.
Copy link
Contributor

Choose a reason for hiding this comment

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

havent found any reference of FEATURE_VIDEOCONFERENCE_WAITING_ROOM_ENABLED
neither client nor server. What this setting is supposed to do then?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good question, i only move the documentations and have the info from yesterday that all of it should be up to date.
Is it is possible to speak with @MartinSchuhmacher to check the details?

Copy link
Contributor

Choose a reason for hiding this comment

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

FEATURE_VIDEOCONFERENCE_WAITING_ROOM_ENABLED is used within handlebars form-create-videoconference.hbs and teams.js controller to determine the options being shown to the user within the pop up window for creating the conference.


Add "videoconference" to School features (table school) in MongoDB.

Make sure that the fields below exists in the videoconference ltitool object (MongoDB).
Copy link
Contributor

Choose a reason for hiding this comment

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

Could we be more comprehensive, what are the necessary steps to take?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I must put the ball back to you. I have no idea, only move the existing documentation and cleanup obvious points. :D
@MartinSchuhmacher

Copy link
Contributor

Choose a reason for hiding this comment

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

What is actually meant by this whole area? in my understanding the videoconference is saved within the videoconferences collection and the targetModel describes if its meant for "courses" or "events" (teams) or "video-conference-elements". So this area should definitely be refactored or I missunderstood something here.
@CeEv

@CeEv CeEv merged commit 3adb632 into main Jan 15, 2026
1 check passed
@CeEv CeEv deleted the add_bbb branch January 15, 2026 13:45
@CeEv CeEv mentioned this pull request Jan 16, 2026
3 tasks

> FEATURE_VIDEOCONFERENCE_ENABLED=true
> VIDEOCONFERENCE_HOST=https://bbb.staging.messenger.schule/bigbluebutton
> VIDEOCONFERENCE_SALT
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 directly add "(from 1Password)" here if we tell the user already what the other values are.
Also add the note to maybe set FEATURE_COLUMN_BOARD_VIDEOCONFERENCE_ENABLED=true to make the option visible on boards.

@MartinSchuhmacher
Copy link
Contributor

follow up comments will be adjusted within PR #84

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