Skip to content

add offramp address and plugin type for the ccip provider#1523

Merged
archseer merged 3 commits intomainfrom
jh/adding-offramp-addr-and-plugin-type-for-ccip-provider-args
Sep 4, 2025
Merged

add offramp address and plugin type for the ccip provider#1523
archseer merged 3 commits intomainfrom
jh/adding-offramp-addr-and-plugin-type-for-ccip-provider-args

Conversation

@huangzhen1997
Copy link
Copy Markdown
Contributor

@huangzhen1997 huangzhen1997 commented Sep 3, 2025

Adding two more field to the CCIP provider args to support contract transmitter instantiation in chainlink core oracle creator plugin

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Sep 3, 2025

apidiff results - no incompatible changes detected ✅

Module: github-com-smartcontractkit-chainlink-common

Status: ✅ 2 compatible changes

(Full summary: https://github.com/smartcontractkit/chainlink-common/actions/runs/17445834418/#summary-49539977918)

@huangzhen1997 huangzhen1997 marked this pull request as ready for review September 3, 2025 20:36
@huangzhen1997 huangzhen1997 requested review from a team as code owners September 3, 2025 20:36
@ogtownsend
Copy link
Copy Markdown
Contributor

can you update CcipProviderArgs usages here too? https://github.com/smartcontractkit/chainlink-common/blob/main/pkg/loop/internal/relayer/relayer.go

@ogtownsend
Copy link
Copy Markdown
Contributor

ogtownsend commented Sep 3, 2025

and one nit if it doesn't cause too much messiness in other locations, could you update rargs CCIPProviderArgs to pargs CCIPProviderArgs or cargs CCIPProviderArgs, ex: https://github.com/smartcontractkit/chainlink-common/blob/main/pkg/types/relayer.go#L258

@archseer archseer enabled auto-merge (squash) September 3, 2025 21:05
@archseer archseer merged commit 17f17c8 into main Sep 4, 2025
20 of 22 checks passed
@archseer archseer deleted the jh/adding-offramp-addr-and-plugin-type-for-ccip-provider-args branch September 4, 2025 13:30
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