Skip to content

Added callback examples #1206

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jun 15, 2017
Merged

Added callback examples #1206

merged 3 commits into from
Jun 15, 2017

Conversation

fehguy
Copy link
Contributor

@fehguy fehguy commented Jun 15, 2017

Fixes #1104 with style, grace, clarity, efficiency and good humor

@fehguy fehguy added this to the v3.0.0-rc2 milestone Jun 15, 2017
format: uri
example: https://tonys-server.com
responses:
201:
Copy link
Member

Choose a reason for hiding this comment

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

statusCodes must be quoted in yaml.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, done

@fehguy
Copy link
Contributor Author

fehguy commented Jun 15, 2017

FYI hyperlink won't work until merged

onData:
# when data is sent, it will be sent to the `callbackUrl` provided
# when making the subscription PLUS the suffix `/data`
$request.query.callbackUrl/data:
Copy link
Contributor

Choose a reason for hiding this comment

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

@darrelmiller 's changes (#1103) use { } around expressions -

myWebhook:
  'http://notificationServer.com?transactionId={$request.body#/id}&email={$request.body#/email}}':

should callbacks likewise embed expressions in {}

@RobDolinMS RobDolinMS merged commit b667bcf into OpenAPI.next Jun 15, 2017
@RobDolinMS RobDolinMS deleted the issue-1104 branch June 15, 2017 16:16
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.

5 participants