Skip to content

Clarify intended behavior on void intent results#1004

Merged
kriswest merged 3 commits intomasterfrom
1003-clarify-getresult-on-resolve-to-void
May 25, 2023
Merged

Clarify intended behavior on void intent results#1004
kriswest merged 3 commits intomasterfrom
1003-clarify-getresult-on-resolve-to-void

Conversation

@kriswest
Copy link
Copy Markdown
Contributor

resolves #1003

Clarifies description of IntentResolution.getResult() when the intent handler returned void (which is not an error)

@kriswest kriswest added bug Something isn't working docs Documentation api FDC3 API Working Group labels May 24, 2023
@kriswest kriswest requested review from a team May 24, 2023 17:22
@netlify
Copy link
Copy Markdown

netlify Bot commented May 24, 2023

Deploy Preview for fdc3 ready!

Name Link
🔨 Latest commit afb80f6
🔍 Latest deploy log https://app.netlify.com/sites/fdc3/deploys/646f7b23edfc4e00086684a8
😎 Deploy Preview https://deploy-preview-1004--fdc3.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 site settings.

Comment thread src/api/DesktopAgent.ts Outdated
@kriswest kriswest mentioned this pull request May 25, 2023
27 tasks
Co-authored-by: Hugh Troeger <htroeger@factset.com>
@kriswest kriswest merged commit 855e34c into master May 25, 2023
@kriswest kriswest deleted the 1003-clarify-getresult-on-resolve-to-void branch May 25, 2023 15:17
@bingenito bingenito mentioned this pull request Nov 6, 2023
18 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api FDC3 API Working Group bug Something isn't working docs Documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clarify expected behaviour of IntentResolution.getResult() when IntentHandlers resolve to void

3 participants