Skip to content

feat: CP-13148 change powered by label#778

Merged
meeh0w merged 2 commits intomainfrom
feat/CP-13148_swap-powered-label
Feb 18, 2026
Merged

feat: CP-13148 change powered by label#778
meeh0w merged 2 commits intomainfrom
feat/CP-13148_swap-powered-label

Conversation

@vvava
Copy link
Contributor

@vvava vvava commented Feb 17, 2026

Description

https://ava-labs.atlassian.net/browse/CP-13148

Changes

Check the network and change the label based on that.

Testing

Go to swap -> change the from token

Screenshots:

Screen.Recording.2026-02-17.at.18.19.39.mov

Checklist for the author

Tick each of them when done or if not applicable.

  • I've covered new/modified business logic with Jest test cases.
  • I've tested the changes myself before sending it to code review and QA.

Copy link
Member

@meeh0w meeh0w left a comment

Choose a reason for hiding this comment

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

You're extracting fromToken.coreChainId from useSwapState() inside of Swap.tsx file just to pass it down to SwapProviderNotice as a prop.

Instead, please take this value from the useSwapState() hook which is already called inside of SwapProviderNotice.

Additionally, you're referencing a useIsAnyAvalancheNetwork() hook from another component. This cross-referencing will make it harder to refactor these components later on (since their files are now depending on each other while they don't need to).

Please either move that hook out of the TokenSelect's directory, or simply use isAvalancheChainId() util from @core/common (the current Swap only supports C-Chain, so no need to match against X/P chains here).

Copy link
Member

@meeh0w meeh0w left a comment

Choose a reason for hiding this comment

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

Addi

@meeh0w meeh0w merged commit d92631b into main Feb 18, 2026
4 checks passed
@meeh0w meeh0w deleted the feat/CP-13148_swap-powered-label branch February 18, 2026 17:37
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

Comments