Skip to content

Adds a new multi-http endpoint to generic-api#4568

Merged
mohamed-mehany merged 5 commits intomainfrom
generic-opendelta
Feb 3, 2026
Merged

Adds a new multi-http endpoint to generic-api#4568
mohamed-mehany merged 5 commits intomainfrom
generic-opendelta

Conversation

@mohamed-mehany
Copy link
Collaborator

@mohamed-mehany mohamed-mehany commented Feb 2, 2026

Closes OPDATA-5695

Description

Adds a new multi-http endpoint that extracts multiple fields from API responses via a dataPaths array.
Supports ripcord-based error handling, designed for V9 schema SmartData Streams.

Changes

  • Adds a new multi-http endpoint

Steps to Test

  1. Local curl tests
  2. Integration tests

Quality Assurance

  • If a new adapter was made, or an existing one was modified so that its environment variables have changed, update the relevant infra-k8s configuration file.
  • If a new adapter was made, or an existing one was modified so that its environment variables have changed, update the relevant adapter-secrets configuration file.
  • If a new adapter was made, or a new endpoint was added, update the test-payload.json file with relevant requests.
  • The branch naming follows git flow (feature/x, chore/x, release/x, hotfix/x, fix/x) or is created from Jira.
  • This is related to a maximum of one Jira story or GitHub issue.
  • Types are safe (avoid TypeScript/TSLint features like any and disable, instead use more specific types).
  • All code changes have 100% unit and integration test coverage. If testing is not applicable or too difficult to justify doing, the reasoning should be documented explicitly in the PR.

@changeset-bot
Copy link

changeset-bot bot commented Feb 2, 2026

🦋 Changeset detected

Latest commit: 38e3604

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@chainlink/generic-api-adapter Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@mohamed-mehany mohamed-mehany marked this pull request as ready for review February 2, 2026 17:07
@mxiao-cll
Copy link
Contributor

What's the points of having this new endpoint vs just use the existing endpoint?

@mohamed-mehany
Copy link
Collaborator Author

What's the points of having this new endpoint vs just use the existing endpoint?

The existing endpoint supports only a single value result.
Also, discussed extending the existing endpoint with the team, and we thought it would be cleaner to have a separate endpoint.

@mxiao-cll mxiao-cll requested review from a team and dskloetc February 3, 2026 14:18
dskloetc
dskloetc previously approved these changes Feb 3, 2026
Copy link
Contributor

@dskloetc dskloetc left a comment

Choose a reason for hiding this comment

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

Discussed with Michael

@mohamed-mehany mohamed-mehany merged commit bdb27c1 into main Feb 3, 2026
16 checks passed
@mohamed-mehany mohamed-mehany deleted the generic-opendelta branch February 3, 2026 17:28
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.

3 participants