Add Speaker to Webinar

[SailReal]

No description provided.

[coderabbitai]

Walkthrough

The PR adds speaker display support to the webinar feature. In assets/js/webinar.js, the $.getJSON success handler is extended to extract speakerName, speakerTitle, and speakerImage from the d.speaker payload field. Two i18n files (i18n/en.yaml and i18n/de.yaml) receive a new webinar_speaker_label entry. The layouts/webinar/single.html template gains a new sidebar <li> block that conditionally renders the speaker's avatar image, name, and optional title when both speakerName and speakerImage are present.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Possibly related PRs

• cryptomator/cryptomator.github.io#163: Introduced the initial webinar feature that this PR extends with speaker field extraction and sidebar rendering.

Suggested reviewers

• tobihagemann

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	No description was provided by the author, making it impossible to evaluate whether the description relates to the changeset.	Add a pull request description explaining the motivation, changes, and impact of adding speaker support to the webinar feature.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'Add Speaker to Webinar' directly reflects the main objective of this PR, which adds speaker data extraction and display functionality across multiple files.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feature/add-speaker-to-webinar

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@assets/js/webinar.js`:
- Around line 12-14: Add a guard check before dereferencing the speaker object
in the data assignment block. The properties d.speaker.name, d.speaker.title,
and d.speaker.avatarUrl are being accessed without verifying that d.speaker
exists, which will throw an error and abort the .done(...) handler if the
speaker payload is absent. Check if d.speaker is defined and truthy before
attempting to access its properties to safely assign values to data.speakerName,
data.speakerTitle, and data.speakerImage.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 513d5f3e-fc66-4b00-872c-9ab002ec0b97

📥 Commits

Reviewing files that changed from the base of the PR and between 026e78f and f9e0b61.

📒 Files selected for processing (4)

• assets/js/webinar.js
• i18n/de.yaml
• i18n/en.yaml
• layouts/webinar/single.html