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 - January 27, 2022 #568

Closed
9 of 10 tasks
kriswest opened this issue Jan 24, 2022 · 24 comments
Closed
9 of 10 tasks

Standard WG Meeting - January 27, 2022 #568

kriswest opened this issue Jan 24, 2022 · 24 comments
Labels
indexed When a meeting attendance is being tracked meeting Standard WG Meeting

Comments

@kriswest
Copy link
Contributor

kriswest commented Jan 24, 2022

Date

Thursday 27 Jan 2022 - 10am EST / 3pm GMT

WebEx info

  • Meeting Link
    Meeting number: 665 568 411
    Password: pbMT7rxay24

More ways to join

  • Join by video system:
  • Join by phone
    • +1-415-655-0003 US Toll
    • +44-20319-88141 UK Toll
  • Access code: 665 568 411

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 legal@finos.org 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.

Agenda

Minutes

  • Checking consensus on the merging of proposals from the Channels, Feeds & Transactions discussion group

    • A brief overview of the process for making additions to the FDC3 standard was provided (
    • No objections were raised to any of the 8 proposals reviewed and consensus was achieved that they should be merged after the reviews by the fdc3 maintainers and editors are completed.
    • However, comments were submitted on two of the proposals :
      • Make it possible to resolve intent based on output type as well as input #498
        • PR: 498 Allow intents to be resolved on output type (where they return data) #499
        • The findIntent functions end up with two optional parameters, does this cause any problems?
          • Should be checked during the maintainers review
          • Types of the arguments are different (Object vs. String) and could be detected. Otherwise one would have to become required, which is undesirable (as currently only the intent is required).
      • Allow intents to be targeted at specific instances of apps #450
        • PR: 450 allow intents to target app instances #509
        • How are failures to deliver an intent dealt with? Should this be an issue to examine if we start defining certification tests for apps?
          • @kriswest There is not currently an error in the ResolveError enumeration to represent the failure to deliver an intent to a selected (or specified) target app or instance - which applies equally to both existing raiseIntent functionality (where an app is launched) and to this case where it is delivered to an app instance - hence its not an issue with this proposal.
            • This should be raised as a separate issue as it represents a shortcoming in FDC3 - when an intent is raised the resolver has to launch an app then wait for it to add the intent handler necessary to deliver the intent to it, if that never happens there is currently no feedback to the app raising the intent (it would receive an intent resolution but not an error).
            • If compliance testing was to confirm that an app does handle intents it claims to, we would need to ensure that an appD record is always included with it and examined as part of the testing.
  • A brief introduction to Support propagation of originating application identity #569 (and Require propagation of originating application identity on context/intent handlers #520) was provided by @bingenito, along with a use case/reason why this issue can prevent firms from being able to use FDC3

    • Discussion to be held at next SWG meeting
  • AppD

    • A brief overview of existing and recently raised issues related to the AppD Standard was provided
    • The formation of a Discussion group to work on the issues raised was proposed
      • Proposal was supported by a number of participants
      • Group to meet fortnightly
      • Group to start with examining role/value/necessity of the AppD as part of the FDC3 specification and a compliance requirement for use of the FDC3 API
      • @kriswest to post poll for dates - not a Thursday due to clashes with other groups

Decisions Made

  • Channels, Feeds & Transactions proposals to be merged into next pre-draft of FDC3 standard (after review by the FDC3 maintainers are complete) 🎉
  • Discussion group focused on AppD to be formed

Action Items

Untracked attendees

Full name Affiliation GitHub username
@Qiana-Citi
Copy link

We've gone through the deck that Kris presented on the proposed API changes and are happy to find that most of the changes are backward compatible. Only the following two items that might have some impact in a way, but that should be acceptable.

fdc3.addIntentListener(), fdc3.addContextListener() & channel.addContextListener()
▪ Change return type from Listener to promise
▪ Used only in the minority of implementations, hence, the change will not affect most implementations.

fdc3.getInfo()
▪ Change return type from ImplementationMetadata to promise
▪ new in 1.2 and usage is not yet widespread.

@bingenito
Copy link
Member

Hi @kriswest, I just created the discussion item - #569. I'd like the opportunity in the agenda this week for about a minute to raise awareness for review and get on agenda in a future meeting to discuss in more detail.

@kriswest
Copy link
Contributor Author

@bingenito I've added it in (sans discussion as we likely won't have time for it) along with an action item to add it to the next meeting agenda. I've also linked to a near identical issue raised recently by @symphony-adnane.

@rikoe
Copy link
Contributor

rikoe commented Jan 27, 2022

I can't attend the first part of today's meeting, but I just want to add my vote for the current channel, feeds and transactions PRs - great work everyone who have gotten us this far.

@mattjamieson
Copy link
Contributor

🎹

@Julia-Ritter
Copy link
Contributor

Julia / FINOS

@bertrand-s
Copy link
Contributor

Bertrand / Symphony

@ggeorgievx
Copy link
Member

Present ✅

@kriswest
Copy link
Contributor Author

Kris / Cosaic

@opoupeney
Copy link

Olivier / Symphony

@bingenito
Copy link
Member

Brian / Morgan Stanley

@ColinEberhardt
Copy link
Contributor

here 👍

@pbaize
Copy link

pbaize commented Jan 27, 2022

Pierre / OpenFin

@mazydar
Copy link

mazydar commented Jan 27, 2022

Mazy @ OpenFin

@timjenkel
Copy link

Tim Jenkel / Wellington

@hampshan
Copy link

Andrew Hampshire present

@symphony-adnane
Copy link

Adnane @ Symphony

@agitana
Copy link
Member

agitana commented Jan 27, 2022

Aitana Myohl / FINOS

@lspiro-Tick42
Copy link

Leslie / Tick42

@RandallDang-Citi
Copy link

Randall / Citi

@MichaelMCoates
Copy link

Michael / OpenFin

@jgavronsky
Copy link

Jane @ FINOS

@hughtroeger
Copy link
Contributor

Hugh / FactSet
(sorry forgot to comment yesterday)

@kriswest
Copy link
Contributor Author

A poll has been posted for the first meeting of the App Directory discussion group, please provide your feedback on dates by visiting #573 (comment) and clicking on the emojis for times that work for you.

@github-actions github-actions bot added the indexed When a meeting attendance is being tracked label Feb 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
indexed When a meeting attendance is being tracked meeting Standard WG Meeting
Projects
None yet
Development

No branches or pull requests