Skip to content

Add Natwest Group and its application details.#1709

Merged
kriswest merged 8 commits intofinos:mainfrom
nikhilagrawalnatwest:feature/Natwest
Feb 9, 2026
Merged

Add Natwest Group and its application details.#1709
kriswest merged 8 commits intofinos:mainfrom
nikhilagrawalnatwest:feature/Natwest

Conversation

@nikhilagrawalnatwest
Copy link
Copy Markdown
Contributor

@nikhilagrawalnatwest nikhilagrawalnatwest commented Nov 25, 2025

Describe your change

Add Natwest Group and its applications information in FDC3 FINOS community page as adopters of FDC3 and as application providers (Agile Markets).

Contributor License Agreement

  • I acknowledge that a contributor license agreement is required and that I have one in place or will seek to put one in place ASAP.

Review Checklist

  • Issue: If a change was made to the FDC3 Standard, was an issue linked above?
  • CHANGELOG: Is a CHANGELOG.md entry included?
  • API changes: Does this PR include changes to any of the FDC3 APIs (DesktopAgent, Channel, PrivateChannel, Listener, Bridging)?
    • Docs & Sources: If yes, were both documentation (/docs) and sources updated?

      JSDoc comments on interfaces and types should be matched to the main documentation in /docs
    • Conformance tests: If yes, are conformance test definitions (/toolbox/fdc3-conformance) still correct and complete?

      Conformance test definitions should cover all required aspects of an FDC3 Desktop Agent implementation, which are usually marked with a MUST keyword, and optional features (SHOULD or MAY) where the format of those features is defined
    • Schemas: If yes, were changes applied to the Bridging and FDC3 for Web protocol schemas?

      The Web Connection protocol and Desktop Agent Communication Protocol schemas must be able to support all necessary aspects of the Desktop Agent API, while Bridging must support those aspects necessary for Desktop Agents to communicate with each other
      • If yes, was code generation (npm run build) run and the results checked in?

        Generated code will be found at /src/api/BrowserTypes.ts and/or /src/bridging/BridgingTypes.ts
  • Context types: Were new Context type schemas created or modified in this PR?
    • Were the field type conventions adhered to?
    • Was the BaseContext schema applied via allOf (as it is in existing types)?
    • Was a title and description provided for all properties defined in the schema?
    • Was at least one example provided?
    • Was code generation (npm run build) run and the results checked in?

      Generated code will be found at /src/context/ContextTypes.ts
  • Intents: Were new Intents created in this PR?

@nikhilagrawalnatwest nikhilagrawalnatwest requested a review from a team as a code owner November 25, 2025 17:36
@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented Nov 25, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 25, 2025

Deploy Preview for fdc3 ready!

Name Link
🔨 Latest commit d2ab5d8
🔍 Latest deploy log https://app.netlify.com/projects/fdc3/deploys/698a27a7ce55000008331e2b
😎 Deploy Preview https://deploy-preview-1709.preview-fdc3.finos.org
📱 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 project configuration.

@mcleo-d
Copy link
Copy Markdown
Member

mcleo-d commented Nov 27, 2025

/easycla

kriswest
kriswest previously approved these changes Feb 9, 2026
Copy link
Copy Markdown
Contributor

@kriswest kriswest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Logo addition looks good to me under both adoptors and app providers:

https://deploy-preview-1709.preview-fdc3.finos.org/community

Can I get another approval from @finos/fdc3-maintainers or @robmoffat please?

kriswest
kriswest previously approved these changes Feb 9, 2026
Copy link
Copy Markdown
Contributor

@kriswest kriswest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still LGTM (after switching an image to .png format)

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds NatWest Group (as an adopter) and Agile Markets (as an application provider) to the FDC3 website’s community/user listings, along with a new Agile Markets logo.

Changes:

  • Add NatWest Group to website/data/users.json (pinned member entry).
  • Add NatWest Group + Agile Markets entries to website/data/community.json.
  • Add AgileMarkets.png logo asset.

Reviewed changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated 7 comments.

File Description
website/static/img/users/AgileMarkets.png Adds the Agile Markets logo used by the new community entry.
website/data/users.json Adds NatWest Group to the users list (caption/image/link metadata).
website/data/community.json Adds NatWest Group (adopter) and Agile Markets (application-provider) entries.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread website/data/community.json
Comment thread website/data/community.json Outdated
Comment thread website/data/community.json
Comment thread website/data/community.json
Comment thread website/data/users.json
Comment thread website/data/users.json Outdated
Comment thread website/data/community.json
bingenito
bingenito previously approved these changes Feb 9, 2026
@nikhilagrawalnatwest nikhilagrawalnatwest deleted the feature/Natwest branch February 9, 2026 17:23
@nikhilagrawalnatwest nikhilagrawalnatwest restored the feature/Natwest branch February 9, 2026 17:53
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@kriswest kriswest self-requested a review February 9, 2026 18:34
Copy link
Copy Markdown
Contributor

@kriswest kriswest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All suggested changes were applied. LGTM

@kriswest kriswest merged commit 3ee24ab into finos:main Feb 9, 2026
7 checks passed
@nikhilagrawalnatwest nikhilagrawalnatwest deleted the feature/Natwest branch February 14, 2026 06:25
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 this pull request may close these issues.

5 participants