Skip to content

Add Callbacks, Links and Webhooks pages#90

Merged
miqui merged 9 commits intoOAI:mainfrom
SensibleWood:main
Apr 3, 2026
Merged

Add Callbacks, Links and Webhooks pages#90
miqui merged 9 commits intoOAI:mainfrom
SensibleWood:main

Conversation

@SensibleWood
Copy link
Copy Markdown
Collaborator

This work is connected to #76

Summary of changes:

  • Added page, diagram and example for Callback object.
  • Added page, diagram and example for Link object (not in scope of referenced ticket, but hey-ho).
  • Added page, diagram and example for webhooks property.
  • Corrected the security examples I got wrong when I wrote the security page (slap on the wrist!)
  • Some typos/syntax things I saw whilst doing this.

👍

@miqui
Copy link
Copy Markdown
Contributor

miqui commented Mar 25, 2024

thanks @SensibleWood - going through this one.

@SensibleWood
Copy link
Copy Markdown
Collaborator Author

thanks @SensibleWood - going through this one.

You the man @miqui

Comment thread specification/callbacks.md
Comment thread specification/webhooks.md Outdated
Copy link
Copy Markdown
Contributor

@lornajane lornajane left a comment

Choose a reason for hiding this comment

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

A few comments, but this is coming together IMO.

Comment thread specification/security.md Outdated
Comment thread specification/webhooks.md Outdated
Comment thread specification/callbacks.md Outdated
@lornajane lornajane requested a review from LasneF June 11, 2024 08:15
Comment thread examples/v3.1/tictactoe.yaml
@lornajane
Copy link
Copy Markdown
Contributor

@SensibleWood I still think this would be really valuable if we could add it. Could you update the branch and let's try to publish? (or let me know and I can pick this up)

@SensibleWood
Copy link
Copy Markdown
Collaborator Author

@lornajane sorry this dropped off my list of things to do. I'll try and get it done before Xmas as the goose is currently too fat for me to fit it in immediately

Comment thread specification/webhooks.md Outdated
@baywet
Copy link
Copy Markdown
Member

baywet commented Jan 29, 2026

@hellobudha can you please follow up on this one for either:

  • merge conflicts + address comments to merge
  • salvage the parts we're interested in
  • hand off to somebody else like @handrews

@lornajane
Copy link
Copy Markdown
Contributor

This is too good to not merge so I've rebased the branch and tried to distill the feedback we had so far into it. Would love another quick check by someone if we can!

@miqui miqui merged commit 32cb567 into OAI:main Apr 3, 2026
2 checks passed
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.

6 participants