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

Standard WG Meeting - August 22nd, 2024 #1317

Closed
9 of 15 tasks
kriswest opened this issue Aug 22, 2024 · 8 comments
Closed
9 of 15 tasks

Standard WG Meeting - August 22nd, 2024 #1317

kriswest opened this issue Aug 22, 2024 · 8 comments

Comments

@kriswest
Copy link
Contributor

kriswest commented Aug 22, 2024

Date

Thursday 22nd August 2024 - 10am (US eastern timezone EDT) / 3pm (London, BST)

Zoom info

  • Join Zoom Meeting
  • Meeting ID: 969 4029 4948
  • Passcode: 636931
  • Dial-in:
    Country International Dial-in Toll-free Dial-in
    US +1 929 205 6099 (New York) 877 853 5247
    UK +44 330 088 5830 0800 031 5717
    France +33 1 8699 5831 0 800 940 415
    Find your local number https://zoom.us/u/ad2WVnBzb8

Meeting notices

  • FINOS Project leads are responsible for observing the FINOS guidelines for running project meetings. Project maintainers can find additional resources in the FINOS Maintainers Cheatsheet.

  • All participants in FINOS project meetings are subject to the LF Antitrust Policy, the FINOS Community Code of Conduct and all other FINOS policies.

  • FINOS meetings involve participation by industry competitors, and it is the intention of FINOS and the Linux Foundation to conduct all of its activities in accordance with applicable antitrust and competition laws. It is therefore extremely important that attendees adhere to meeting agendas, and be aware of, and not participate in, any activities that are prohibited under applicable US state, federal or foreign antitrust and competition laws. Please contact [email protected] with any questions.

  • FINOS project meetings may be recorded for use solely by the FINOS team for administration purposes. In very limited instances, and with explicit approval, recordings may be made more widely available.

Participation Requirements

Note: Meeting participants are expected to accept the terms of the FDC3 license (Community Specification License), understand the governance process and have a CLA in place.

Please click the following links at the start of the meeting if you have not done so previously.

Tracking Attendance

Note: Meeting participants are expected to add a comment to this GitHub issue in order that we can track attendance of FDC3 project meetings. Please do this at the start of the meeting.

Agenda

Minutes

  • Announcements

    • Status of FDC3 for the Web
      • Intending to raise the completed protocol and software addition as [@experimental}(https://fdc3.finos.org/docs/fdc3-compliance#experimental-features) - but expecting to remove the label quickly as there are already 1 OS implementation and 2 commercial implementations in the works.
    • Generated context docs
      • Checkout the new context documentation, generated from the context schemas, which are now the single source of truth for the context standardisation, at: https://fdc3.finos.org/docs/next/context/ref/Chart
        • Currently only available in the Main/Unreleased section of the swebsite (until we release a 2.2 version): image
    • FDC3 for .NET docs
      • Will be merged and available in Main/Unreleased shortly (pending final editors review).
    • FDC3 monorepo #1297
      • Work from a contributor is ongoing that will convert the FDC3 repo into a true mono-repo to solve a number issues and make it easier to publish reference implementations.
      • There was some discussion of the best order of operations (merging in-flight PRs, including FDC3 for the web and rebasing the mono-repo refactor or vice-versa). If possible, getting the mono-repo refactor in first would be a good idea.
  • Inconsistent Use of Promises #1294 & Make Listener.unsubscribe() async for consistency #1300

    • @kriswest provided some background on the work in 2.0 to make the API all async (and PrivateChannel/Listener functions having been missed out) and the recent addition of the addEventListener function to the desktop agent.
    • It was proposed to make additive changes to API (make Listener.unsubscribe and PrivateChannel.disconnect() return Promise<void>, deprecate synchronous PrivateChannel event listner functions and add a new async addEventListener function to replace them in the same style used in the DesktopAgent` interface, for consistency
    • The PR also changes the event type enum(s) introduced in Add event listener support to the Desktop Agent API #1207 with string unions as this allows for a better inheritane structure in the event objects and event handler, that is easier to implement in .NET.
    • The PR was reviewed in the meeting and consent sought and received to merge the changes, with one minor changes: replace "USER_CHANNEL_CHANGED" with "userChannelChanged" as the desktop agent event type for consistency with the PrivateChannel event types (e.g. "addContextListener"
  • Question: Disposal/Teardown patterns in FDC3 #1263

    • There was some discussion of background to this issue (raised by @Davidhanson90), including how containers (the traditional focus of FDC3) haven't needed any explicit tear-down calls as they tend to create connections on the opening of a window and then clean up when the observe it closing.
    • There was consensus that there is not a clear use case for disconnect() call or similar for conventional FDC3 use.
    • However, there is a clear need in web browsers as Desktop Agents can't as easily observe that a window is closed/has closed. It is to listen for a MessagePort closing, but the event may not fire if a render thread crashes, hence, implementations should periodically check the port is still working.
      • A goodbye message should be added to the Web Connection Protocol to handle tear-down and the client code can call it automatically via onBeforeUnload - however implementors would still be advised to retain instance details for apps as they may reconnect quickly, e.g. after a navigation event.
    • Further, Support for Multiple Application Scopes (Agents) within a single window process #1250 is intended to solve for cases where there may be multiple 'apps' or 'widgets' within a single page, and each needing to ahve a distinct identify in FDC3 so that they can interoperate with each other as well as other applications via FDC3.
  • Add property to support analytics across apps #1290

    • @kriswest provided some background on the prior discussion of this issue - there is consensus that its an important usecase, but consent was withheld on the specific proposed solution (adding a metadata property to the context data schema) - it's easy (requires no changes to a DA) but not entirely clean (mixes data and metadata).
    • @kriswest went over an alternative proposal based on adding optional metadata arguments to broadcast and raiseIntent functions which then deliver the metadata in the ContextMetadata object - requires more work from DA vendors, but is much cleaner.
    • The meeting agreed to wait for @Davidhanson90 and @julianna-ciq to meet on the topic and discuss the options - the issue will be added to next month's meeting.
      • The additional arguments could be optional as ContextMetadata is currently an optional feature
      • The change is likely to miss FDC3 2.2 and go into the FDC3 2.3 scope.

Action Items

Rolled over:

Untracked attendees

Full name Affiliation GitHub username
@kriswest kriswest changed the title Standard WG Meeting - August 21st, 2024 Standard WG Meeting - August 22nd, 2024 Aug 22, 2024
@novavi
Copy link

novavi commented Aug 22, 2024

Derek Novavi / S&P Global

@bingenito
Copy link
Member

Brian Ingenito / Morgan Stanley

@openfin-johans
Copy link
Contributor

Johan Sandersson / Here (formerly OpenFin) 🎁

@psmulovics
Copy link
Contributor

Peter Smulovics / Morgan Stanley

@Lecss
Copy link

Lecss commented Aug 22, 2024

Alex Dumitru / Citi

@kriswest
Copy link
Contributor Author

Kris West / interop.io 🚀

@robmoffat
Copy link
Member

Rob Moffat / FINOS 🚚

@lilla28
Copy link

lilla28 commented Aug 22, 2024

Lilla Orban / Morgan Stanley

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

No branches or pull requests

8 participants