Skip to content

Add openrouter conversation component docs#5148

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

Add openrouter conversation component docs#5148
jjcollinge wants to merge 3 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

@msfussell msfussell added the waiting-on-code-pr The code PR needs to be merged before the docs are updated label May 23, 2026
@msfussell msfussell added this to the v1.18 milestone May 23, 2026
@marcduiker marcduiker modified the milestones: v1.18, v1.19 Jun 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

waiting-on-code-pr The code PR needs to be merged before the docs are updated

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants