Skip to content

Add native Videokonferenz API - #51

Open
joan-code6 wants to merge 1 commit into
mainfrom
feat/videokonferenz-api
Open

joan-code6 wants to merge 1 commit into
mainfrom
feat/videokonferenz-api

Conversation

@joan-code6

@joan-code6 joan-code6 commented Sep 14, 2026

Copy link
Copy Markdown
Owner

Summary

  • parse native Schulportal video-room overviews
  • poll the authenticated live room-status endpoint used by Schulportal
  • expose an uncached account-bound API with safe HTTPS links
  • register the module and sidebar preference

Validation

  • 301 passed, 1 skipped
  • Ruff passes for the new module and tests
  • aggregate live calibration: 23 eligible accounts, 23 successful catalogs/status polls, 295 rooms parsed

Paired PR

Rollout

Merge and deploy this backend PR before the paired UI PR.

@coderabbitai

coderabbitai Bot commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 911a27c1-7bcd-4152-9a7f-4a20651e1ae5


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: cf6a152209

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread api/auth_db.py
"search",
"dashboard",
"messages",
"videokonferenz",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Migrate saved sidebar orders to include Videokonferenz

For users who already have preferences stored, _merge_nested_preferences replaces the default sidebar.order list with the saved list, so adding this ID only to DEFAULT_SIDEBAR_ORDER does not add it to their returned preferences. Consequently, /settings/preferences continues to omit videokonferenz until the user explicitly saves a new sidebar order, preventing clients that treat this order as authoritative from showing the new module. Append newly introduced default items while decoding existing preferences, as the update route already does when an order is saved.

Useful? React with 👍 / 👎.

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.

1 participant