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

Update to FDC3 2.1 #74

Closed
17 of 18 tasks
bingenito opened this issue Nov 6, 2023 · 7 comments · Fixed by #141
Closed
17 of 18 tasks

Update to FDC3 2.1 #74

bingenito opened this issue Nov 6, 2023 · 7 comments · Fixed by #141
Assignees

Comments

@bingenito
Copy link
Member

bingenito commented Nov 6, 2023

Checklist extracted from FDC3 CHANGELOG

Added

Changed

Deprecated

Fixed

@bingenito bingenito self-assigned this Nov 6, 2023
@bingenito
Copy link
Member Author

Work will start now that 2.0 has been transferred and republished

@bingenito
Copy link
Member Author

Experimental Order, OrderList, Product, Trade & TradeList context types will be intentionally left out until official (unless requested)

@bingenito
Copy link
Member Author

Intents are still not defined/listed within fdc3-dotnet so any additions to enums/constants are not replicated

@kriswest
Copy link

kriswest commented Aug 1, 2024

Intents are still not defined/listed within fdc3-dotnet so any additions to enums/constants are not replicated

@bingenito This is all we have in TS for intents: https://github.com/finos/FDC3/blob/fdc3-for-web/src/intents/Intents.ts
Theres this JSON file as well (which contains displayNames that should go as they are deprecated). Perhaps you could script something based on the file?

I note the following are missing from the JSON file: SendChatMessage, ViewAnalysis, ViewChat, ViewMessages - I'll drop a quick PR to fix those ommissions

@kriswest
Copy link

kriswest commented Aug 1, 2024

Experimental Order, OrderList, Product, Trade & TradeList context types will be intentionally left out until official (unless requested)

Boo, here I was hoping you'd be helping to drive getting them right 😜

@bingenito
Copy link
Member Author

Intents are still not defined/listed within fdc3-dotnet so any additions to enums/constants are not replicated

@bingenito This is all we have in TS for intents: https://github.com/finos/FDC3/blob/fdc3-for-web/src/intents/Intents.ts Theres this JSON file as well (which contains displayNames that should go as they are deprecated). Perhaps you could script something based on the file?

I note the following are missing from the JSON file: SendChatMessage, ViewAnalysis, ViewChat, ViewMessages - I'll drop a quick PR to fix those ommissions

@kriswest As this was never there for unpublished 1.2 or 2.0, I'll leave out of this PR but will create a separate issue to add an Intents.cs with public static strings.

@kriswest
Copy link

kriswest commented Aug 1, 2024

Added a PR to fix and simplify the JSON file: finos/FDC3#1304

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants