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

Add an error to ResolveError for failure to deliver an intent to an app #581

Closed
kriswest opened this issue Feb 7, 2022 · 0 comments · Fixed by #601
Closed

Add an error to ResolveError for failure to deliver an intent to an app #581

kriswest opened this issue Feb 7, 2022 · 0 comments · Fixed by #601
Labels
api FDC3 API Working Group enhancement New feature or request
Milestone

Comments

@kriswest
Copy link
Contributor

kriswest commented Feb 7, 2022

Enhancement Request

Add a IntentDeliveryFailed error to the ResolveError enumeration.

Use Case:

The ResolveError enumeration provides standard string for failures you might get back when raising an intent or findIntent runs into a problem (e.g. the are no apps available for that intent, the resolver isn't available or the user cancelled resolution). However, it doesn't include a standardized error for cases where it succeeded in resolving to an app or app instance, but couldn't deliver the intent and context to it. This might occur because:

  1. the app failed to add an intent listener or,
  2. added an intent listener but not for the right type
  3. or failed in some other way (e.g. it closed immediately).

Additional Information

There is some overlap with the errors added in the recently merged PR for targetting app instances:

However, these are intended to indicate that a specifically targetted app or instance is not available to be started or has been closed (e.g. option for an instance was shown in a resolver, but it closed before delivery or the target app couldn't be started for some reason). This handles case 3 above, but not 1 or 2.

Clear documentation of which error to use when will be added to ensure the various situations are easily differentiated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api FDC3 API Working Group enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant