Skip to content

Commit

Permalink
Merge pull request #1280 from finos/IntentMetadata.displayName-should…
Browse files Browse the repository at this point in the history
…-be-optional

Make IntentMetadata.displayName optional as its deprecated
  • Loading branch information
kriswest authored Jul 23, 2024
2 parents 07714f1 + 760cc25 commit ef3702f
Show file tree
Hide file tree
Showing 6 changed files with 6 additions and 6 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,8 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).

### Deprecated

* Made `IntentMetadata.displayName` optional as it is deprecated. ([#1280](https://github.com/finos/FDC3/pull/1280))

### Fixed

* Added missing `desktopAgent` field to ImplementationMetadata objects returned for all agents connect to a DesktopAgent bridge in Connection Step 6 connectAgentsUpdate messages and refined the schema used to collect this info in step 3 handshake. ([#1177](https://github.com/finos/FDC3/pull/1177))
Expand Down
2 changes: 1 addition & 1 deletion docs/api/ref/Metadata.md
Original file line number Diff line number Diff line change
Expand Up @@ -293,7 +293,7 @@ interface IntentMetadata {
* @deprecated Use the intent name for display as display name may vary for
* each application as it is defined in the app's AppD record.
*/
readonly displayName: string;
readonly displayName?: string;
}
```

Expand Down
1 change: 0 additions & 1 deletion schemas/api/api.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,6 @@
},
"additionalProperties": false,
"required": [
"displayName",
"name"
]
},
Expand Down
2 changes: 1 addition & 1 deletion src/api/IntentMetadata.ts
Original file line number Diff line number Diff line change
Expand Up @@ -16,5 +16,5 @@ export interface IntentMetadata {
* @deprecated Use the intent name for display as display name may vary for
* each application as it is defined in the app's AppD record.
*/
readonly displayName: string;
readonly displayName?: string;
}
4 changes: 2 additions & 2 deletions src/bridging/BridgingTypes.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1769,7 +1769,7 @@ export interface IntentMetadata {
/**
* Display name for the intent.
*/
displayName: string;
displayName?: string;
/**
* The unique name of the intent that can be invoked by the raiseIntent call
*/
Expand Down Expand Up @@ -5206,7 +5206,7 @@ const typeMap: any = {
{ json: "intent", js: "intent", typ: r("IntentMetadata") },
], false),
"IntentMetadata": o([
{ json: "displayName", js: "displayName", typ: "" },
{ json: "displayName", js: "displayName", typ: u(undefined, "") },
{ json: "name", js: "name", typ: "" },
], false),
"FindIntentBridgeErrorResponse": o([
Expand Down
1 change: 0 additions & 1 deletion website/static/schemas/next/api/api.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,6 @@
},
"additionalProperties": false,
"required": [
"displayName",
"name"
]
},
Expand Down

0 comments on commit ef3702f

Please sign in to comment.