Skip to content
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

Make it possible to resolve intent based on output type as well as input #498

Closed
kriswest opened this issue Nov 12, 2021 · 2 comments · Fixed by #508
Closed

Make it possible to resolve intent based on output type as well as input #498

kriswest opened this issue Nov 12, 2021 · 2 comments · Fixed by #508
Labels
channels feeds & transactions Channels, Feeds & Transactions Discussion Group enhancement New feature or request intents
Milestone

Comments

@kriswest
Copy link
Contributor

Enhancement Request

Use Case:

Issue #432 (and associated PR #495) make it possible to return data from a raised intent. However, an application or resolver UI may wish resolve an intent based on the output Context data type, in addition to the input type, ensuring they know what data type to expect.

To do so, additional data representing this information must be added to the AppD's Application schema, while the findIntent and raiseIntent and related functions will need to take extra arguments facilitating the resolution.

Workflow Examples

For example, an application may allow the user to raise the ViewAnalysis intent, but require that data is returned encoding or referencing the analysis for inclusion within notes kept within the application. It either resolves the intent via findIntent followed by a targetted raiseIntent, or calls raiseIntent and relies on a resolver UI for resolution, and passes the extra detail on what type it expects to be returned. Only applications that can return the specified type are offered.

@kriswest kriswest added enhancement New feature or request intents channels feeds & transactions Channels, Feeds & Transactions Discussion Group labels Nov 12, 2021
@kriswest kriswest added this to the 2.0 milestone Nov 12, 2021
@kriswest
Copy link
Contributor Author

This turned out to be difficult to add as an optional argument to the raiseIntent functions but is fine on the findIntent functions. Notes on that from the PR:

Support for this was not added to the raiseIntent APIs as:

  • a second optional string argument would be a problem (that it isn't on findIntent)
  • this form of resolution is likely to be performed programmatically, rather than through a resolver UI

@kriswest
Copy link
Contributor Author

was resolved by #508

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
channels feeds & transactions Channels, Feeds & Transactions Discussion Group enhancement New feature or request intents
Projects
None yet
1 participant