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

Refine the varying schemas for ImplementationMetadata #1177

Merged
merged 5 commits into from
Apr 10, 2024

Conversation

kriswest
Copy link
Contributor

@kriswest kriswest commented Mar 26, 2024

resolves #1057

There are several variations on the ImplementationMetadata schema (in the API it has AppMetadata, in bridging connection step 6 connectedAgentsUpdate its got the name assigned to each agent by the bridge, in connection step 3 it has neither). This PR ensures that the definitions generated for each type are correct. The desktopAgent name was previously missing from the connectedAgentsUpdate, which is a problem it doesn't match what is described in the specification

@kriswest kriswest added api FDC3 API Working Group Desktop Agent Bridging Desktop Agent Bridging Discussion Group labels Mar 26, 2024
Copy link

netlify bot commented Mar 26, 2024

Deploy Preview for fdc3 ready!

Name Link
🔨 Latest commit 5e859ec
🔍 Latest deploy log https://app.netlify.com/sites/fdc3/deploys/6616a46ec4d12b0008b250a9
😎 Deploy Preview https://deploy-preview-1177--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 configuration.

@kriswest kriswest changed the title Refines the varying schemas for ImplementationMetadata Refine the varying schemas for ImplementationMetadata Mar 26, 2024
@kriswest kriswest requested review from a team and robmoffat March 26, 2024 16:23
schemas/bridging/common.schema.json Outdated Show resolved Hide resolved
Co-authored-by: Hugh Troeger <htroeger@factset.com>
@kriswest kriswest requested a review from a team April 5, 2024 11:21
@kriswest kriswest merged commit 02a43a5 into main Apr 10, 2024
7 checks passed
kriswest added a commit that referenced this pull request Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api FDC3 API Working Group Desktop Agent Bridging Desktop Agent Bridging Discussion Group
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing desktopAgent from ImplementationMetadata
2 participants