Skip to content

docs(calendar): document setting meeting owner via full API#1903

Merged
calendar-assistant merged 1 commit into
mainfrom
docs/calendar-meeting-owner
Jul 15, 2026
Merged

docs(calendar): document setting meeting owner via full API#1903
calendar-assistant merged 1 commit into
mainfrom
docs/calendar-meeting-owner

Conversation

@calendar-assistant

@calendar-assistant calendar-assistant commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

When creating a calendar event, if the caller wants to set the meeting owner, the +create shortcut does not expose this field. This documents how to set it via the full API command so AI agents route to the correct path.

Changes

  • Add a note under the "key differences of the full API command" list in lark-calendar-create.md: set the meeting owner via vchat.meeting_settings.owner_id, which requires vchat.vc_type=vc, is effective only for app (bot) identity operating on an app calendar, and the owner must be a user (ou_ open_id), not a non-user or external-tenant user.

Test Plan

  • Docs-only change; no code paths affected
  • Unit tests pass (N/A — documentation only)

Related Issues

  • None

Summary by CodeRabbit

  • Documentation
    • Clarified that approved meeting rooms cannot have an owner set through the +create API.
    • Added guidance for configuring a meeting owner through the full API, including required meeting type, application calendar context, and eligible user identity requirements.

@github-actions github-actions Bot added domain/calendar PR touches the calendar domain size/M Single-domain feat or fix with limited business impact labels Jul 15, 2026
@coderabbitai

coderabbitai Bot commented Jul 15, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7810113f-e4a4-4aec-94f5-48522eca768d

📥 Commits

Reviewing files that changed from the base of the PR and between 5f0f7a1 and 2ed99c5.

📒 Files selected for processing (1)
  • skills/lark-calendar/references/lark-calendar-create.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • skills/lark-calendar/references/lark-calendar-create.md

📝 Walkthrough

Walkthrough

更新会议室创建 API 差异说明,补充会议 owner 的配置字段、调用身份、日历类型及 owner 标识限制。

Changes

日历 API 文档

Layer / File(s) Summary
会议 Owner 配置说明
skills/lark-calendar/references/lark-calendar-create.md
说明通过完整 API 设置 vchat.meeting_settings.owner_id,同时将 vchat.vc_type 设为 vc,并限定 bot 操作应用日历及用户 ou_ open_id。

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly reflects the documentation change about setting meeting owners through the full API.
Description check ✅ Passed The description follows the template and includes summary, changes, test plan, and related issues.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/calendar-meeting-owner

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.

@codecov

codecov Bot commented Jul 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.72%. Comparing base (8897196) to head (2ed99c5).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1903   +/-   ##
=======================================
  Coverage   74.72%   74.72%           
=======================================
  Files         886      886           
  Lines       92621    92621           
=======================================
  Hits        69208    69208           
  Misses      18052    18052           
  Partials     5361     5361           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

github-actions Bot commented Jul 15, 2026

Copy link
Copy Markdown

🚀 PR Preview Install Guide

🧰 CLI update

npm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@2ed99c5ebc72e11d5db5c7df3ed351bd8542b7bc

🧩 Skill update

npx skills add larksuite/cli#docs/calendar-meeting-owner -y -g

Note that meeting owner must be set via vchat.meeting_settings.owner_id
with vchat.vc_type=vc, effective only for app (bot) identity on app
calendars, since +create does not expose this field.
@calendar-assistant
calendar-assistant force-pushed the docs/calendar-meeting-owner branch from 5f0f7a1 to 2ed99c5 Compare July 15, 2026 11:20
@calendar-assistant
calendar-assistant merged commit 64e10a0 into main Jul 15, 2026
38 checks passed
@calendar-assistant
calendar-assistant deleted the docs/calendar-meeting-owner branch July 15, 2026 11:33
@liangshuo-1 liangshuo-1 mentioned this pull request Jul 15, 2026
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain/calendar PR touches the calendar domain size/M Single-domain feat or fix with limited business impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants