feat: Add BigBlueButton video conferencing integration#28223
feat: Add BigBlueButton video conferencing integration#28223Awesome-wml wants to merge 1 commit intocalcom:mainfrom
Conversation
|
|
There was a problem hiding this comment.
1 issue found across 8 files
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="packages/app-store/bigbluebuttonvideo/package.json">
<violation number="1" location="packages/app-store/bigbluebuttonvideo/package.json:6">
P2: Internal @calcom dependencies should use the workspace protocol ("workspace:*") like other app-store packages. Using "*" can resolve to registry versions instead of local workspace packages, causing version drift in CI/dev.</violation>
</file>
Since this is your first cubic review, here's how it works:
- cubic automatically reviews your code and comments on bugs and improvements
- Teach cubic by replying to its comments. cubic learns from your replies and gets better over time
- Add one-off context when rerunning by tagging
@cubic-dev-aiwith guidance or docs links (includingllms.txt) - Ask questions if you need clarification on any suggestion
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
| @@ -0,0 +1,12 @@ | |||
| { | |||
There was a problem hiding this comment.
P2: Internal @calcom dependencies should use the workspace protocol ("workspace:") like other app-store packages. Using "" can resolve to registry versions instead of local workspace packages, causing version drift in CI/dev.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/app-store/bigbluebuttonvideo/package.json, line 6:
<comment>Internal @calcom dependencies should use the workspace protocol ("workspace:*") like other app-store packages. Using "*" can resolve to registry versions instead of local workspace packages, causing version drift in CI/dev.</comment>
<file context>
@@ -0,0 +1,12 @@
+ "version": "0.0.1",
+ "main": "./index.ts",
+ "dependencies": {
+ "@calcom/lib": "*",
+ "@calcom/types": "*"
+ },
</file context>
|
Hi @PeerRich and @calcom/foundation team,
|
romitg2
left a comment
There was a problem hiding this comment.
@Awesome-wml attach video demo, and pr looks incomplete as i can't see any auth related code.
/claim #1985
Summary by cubic
Adds BigBlueButton as a new video conferencing provider with a dynamic location link and a VideoApiAdapter to create meetings. Implements Linear #1985 by registering the app in the store and wiring up meeting creation.
Written for commit 57d10c4. Summary will update on new commits.