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

450 allow intents to target app instances #509

Merged

Conversation

kriswest
Copy link
Contributor

resolves #450
Adds an optional instanceId field to AppMetadata, allowing it to represent specific instances of applications.

Also updates open() to return AppMetadata (identifying the instance opened). raiseIntent() already returns details of the app resolving an intent, however, this should now always be an AppMetadata, with the optional AppMetadata.instanceId field set, to represent the app instance that received the intent (and may have been started to do so).

@kriswest kriswest added channels feeds & transactions Channels, Feeds & Transactions Discussion Group enhancement New feature or request api FDC3 API Working Group labels Nov 22, 2021
…ications and errors on non-existent target instances
Copy link
Contributor

@thorsent thorsent left a comment

Choose a reason for hiding this comment

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

Great

src/api/AppMetadata.ts Outdated Show resolved Hide resolved
src/api/DesktopAgent.ts Outdated Show resolved Hide resolved
Co-authored-by: Matt Jamieson <10372+mattjamieson@users.noreply.github.com>
@kriswest kriswest merged commit e042101 into finos:master Feb 4, 2022
@kriswest kriswest deleted the 450-allow-intents-to-target-app-instances branch February 4, 2022 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api FDC3 API Working Group channels feeds & transactions Channels, Feeds & Transactions Discussion Group cla-present enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow intents to be targeted at specific instances of apps
5 participants