Skip to content

Update DefaultAccessor GetFeeQuoterTokenUpdates and add SolanaUSDCReaderAccessor impl#1206

Merged
ogtownsend merged 4 commits intomainfrom
ogt/update-GetFeeQuoterTokenUpdates-price-reader
Sep 30, 2025
Merged

Update DefaultAccessor GetFeeQuoterTokenUpdates and add SolanaUSDCReaderAccessor impl#1206
ogtownsend merged 4 commits intomainfrom
ogt/update-GetFeeQuoterTokenUpdates-price-reader

Conversation

@ogtownsend
Copy link
Contributor

@ogtownsend ogtownsend commented Sep 18, 2025 •
edited
Loading

core ref: 32f3b8224406feeee2665bdd5a260fd63a71dca7

Bumping cl-common import and updating default_price_reader:

GetFeeQuoterTokenUpdates(
		ctx context.Context,
		tokens []UnknownEncodedAddress,
		chain ChainSelector,
	) (map[UnknownEncodedAddress]TimestampedUnixBig, error)

updated to:

GetFeeQuoterTokenUpdates(
		ctx context.Context,
		tokensBytes []UnknownAddress,
	) (map[UnknownEncodedAddress]TimestampedUnixBig, error)

Also adds SolanaUSDCReaderAccessor

Related PRs:

  1. Bump cl-common import and update GetFeeQuoterTokenUpdates chainlink-ton#190
  2. CCIP - add ExtraDataCodecBundle protobuf and client/server support chainlink-common#1548
  3. CCIP - add ExtraDataCodecRegistry to sync SourceChainExtraDataCodecs across LOOPed relayers chainlink#19550

@ogtownsend ogtownsend force-pushed the ogt/update-GetFeeQuoterTokenUpdates-price-reader branch 5 times, most recently from 05970d7 to 332dc29 Compare September 19, 2025 01:29
@ogtownsend ogtownsend mentioned this pull request Sep 19, 2025
Closed
@ogtownsend ogtownsend force-pushed the ogt/update-GetFeeQuoterTokenUpdates-price-reader branch from 332dc29 to 60c7fd2 Compare September 19, 2025 20:06
@ogtownsend ogtownsend marked this pull request as ready for review September 19, 2025 20:06
@ogtownsend ogtownsend requested review from a team as code owners September 19, 2025 20:06
This was referenced Sep 19, 2025
Merged
Merged
@ogtownsend ogtownsend marked this pull request as draft September 23, 2025 21:12
@ogtownsend ogtownsend mentioned this pull request Sep 24, 2025
Merged
@ogtownsend ogtownsend force-pushed the ogt/update-GetFeeQuoterTokenUpdates-price-reader branch from ed8659d to 59224fd Compare September 24, 2025 23:53
@ogtownsend ogtownsend force-pushed the ogt/update-GetFeeQuoterTokenUpdates-price-reader branch from 59224fd to f288b74 Compare September 29, 2025 16:54
@ogtownsend ogtownsend marked this pull request as ready for review September 29, 2025 17:03
amit-momin
amit-momin previously approved these changes Sep 29, 2025
View reviewed changes
amit-momin
amit-momin previously approved these changes Sep 29, 2025
View reviewed changes
@ogtownsend ogtownsend changed the title Update DefaultAccessor GetFeeQuoterTokenUpdates Update DefaultAccessor GetFeeQuoterTokenUpdates and add SolanaUSDCReaderAccessor impl Sep 29, 2025
@github-actions
Copy link

github-actions bot commented Sep 30, 2025

Metric ogt/update-GetFeeQuoterTokenUpdates-price-reader main
Coverage 69.3% 68.7%

@ogtownsend ogtownsend added this pull request to the merge queue Sep 30, 2025
Merged via the queue into main with commit 88c08e6 Sep 30, 2025
48 checks passed
@ogtownsend ogtownsend deleted the ogt/update-GetFeeQuoterTokenUpdates-price-reader branch September 30, 2025 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@winder winder winder approved these changes

@matYang matYang matYang approved these changes

@amit-momin amit-momin amit-momin left review comments

@asoliman92 asoliman92 Awaiting requested review from asoliman92 asoliman92 is a code owner automatically assigned from smartcontractkit/ccip-offchain

@agusaldasoro agusaldasoro Awaiting requested review from agusaldasoro agusaldasoro is a code owner automatically assigned from smartcontractkit/ccip-offchain

@mateusz-sekara mateusz-sekara Awaiting requested review from mateusz-sekara mateusz-sekara is a code owner automatically assigned from smartcontractkit/ccip-offchain

@0xAustinWang 0xAustinWang Awaiting requested review from 0xAustinWang 0xAustinWang is a code owner automatically assigned from smartcontractkit/ccip-offchain

@tt-cll tt-cll Awaiting requested review from tt-cll tt-cll is a code owner automatically assigned from smartcontractkit/ccip-offchain

@chris-de-leon-cll chris-de-leon-cll Awaiting requested review from chris-de-leon-cll chris-de-leon-cll is a code owner automatically assigned from smartcontractkit/ccip-offchain

@archseer archseer Awaiting requested review from archseer

@huangzhen1997 huangzhen1997 Awaiting requested review from huangzhen1997

@jadepark-dev jadepark-dev Awaiting requested review from jadepark-dev

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ogtownsend @winder @matYang @amit-momin