Skip to content

Conversation

@ryougi-misaki
Copy link

Add the missing required parameter Role: schema.User to the example code

What type of PR is this?

docs: Documentation only changes

Check the PR title.

  • This PR title match the format: <type>(optional scope): <description>
  • The description of this PR title is user-oriented and clear enough for others to understand.

(Optional) Translate the PR title into Chinese.

docs(eino): 在多模态示例中补充缺失的 Role 字段

(Optional) More detailed description for this PR(en: English/zh: Chinese).

en:
This PR fixes a critical omission in the multimodal chat example code. The schema.Message struct requires the Role field (e.g., schema.User) to be explicitly set, but it was missing in the documentation example. Without it, users would encounter runtime errors when trying to run the sample. This change adds the required Role: schema.User to ensure the example works correctly out of the box.

zh(optional):
本 PR 修复了多模态聊天示例代码中的一个关键遗漏。schema.Message 结构体必须显式设置 Role 字段(例如 schema.User),但文档示例中缺失了该字段,导致用户直接运行示例时会报错。本次修改补充了必要的 Role: schema.User,确保示例代码开箱即用。

(Optional) Which issue(s) this PR fixes:

N/A

Add the missing required parameter Role: schema.User to the example code
@ryougi-misaki ryougi-misaki requested review from a team as code owners January 14, 2026 06:16
@netlify
Copy link

netlify bot commented Jan 14, 2026

Deploy Preview for cloudwego ready!

Name Link
🔨 Latest commit fcf9a72
🔍 Latest deploy log https://app.netlify.com/projects/cloudwego/deploys/696735eb3485d300080cf1f2
😎 Deploy Preview https://deploy-preview-1496--cloudwego.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants