Skip to content

Add openrouter conversation component docs#5148

Open
jjcollinge wants to merge 2 commits into
dapr:v1.18from
jjcollinge:jjcollinge/openrouter-conversation-docs
Open

Add openrouter conversation component docs#5148
jjcollinge wants to merge 2 commits into
dapr:v1.18from
jjcollinge:jjcollinge/openrouter-conversation-docs

Conversation

@jjcollinge
Copy link
Copy Markdown
Contributor

@jjcollinge jjcollinge commented Apr 24, 2026

Thank you for helping make the Dapr documentation better!

Please follow this checklist before submitting:

  • Commits are signed with Developer Certificate of Origin (DCO - learn more)
  • Read the contribution guide
  • Commands include options for Linux, MacOS, and Windows within tabpane
  • New file and folder names are globally unique
  • Page references use shortcodes instead of markdown or URL links
  • Images use HTML style and have alternative text
  • Places where multiple code/command options are given have tabpane

In addition, please fill out the following to help reviewers understand this pull request:

Description

Add docs for dapr/components-contrib#4334

Issue reference

@jjcollinge jjcollinge requested review from a team as code owners April 24, 2026 19:55
@jjcollinge jjcollinge changed the base branch from v1.17 to v1.18 April 24, 2026 19:56
Signed-off-by: Joni Collinge <jonathancollinge@live.com>
@jjcollinge jjcollinge force-pushed the jjcollinge/openrouter-conversation-docs branch from ef0572d to a1bcb45 Compare April 27, 2026 09:44
Copy link
Copy Markdown
Contributor

@nelson-parente nelson-parente left a comment

Choose a reason for hiding this comment

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

Pre-review for @marcduiker handoff: LGTM with nits.

  • New OpenRouter conversation component page looks correct: key metadata fields, cacheTTL, siteURL/siteTitle for attribution, and the data/components YAML entry with since: "1.18".
  • Nit: the DCO checklist items are all unchecked. The author should sign their commits per Dapr contributing guidelines. Marc, worth confirming DCO is satisfied before merging.
  • Nit: model field shows openai/gpt-4o-mini as example in YAML but the description says default is also openai/gpt-4o-mini — consistent, good.
    Nothing blocking if DCO is satisfied — Marc, your call.

Copy link
Copy Markdown
Contributor

@marcduiker marcduiker left a comment

Choose a reason for hiding this comment

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

LGTM

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.

3 participants