Skip to content

feat: add getSwapTokens action#234

Merged
melisaguevara merged 3 commits intomasterfrom
mguevara/acx-4320-implement-gettokensactions
Aug 25, 2025
Merged

feat: add getSwapTokens action#234
melisaguevara merged 3 commits intomasterfrom
mguevara/acx-4320-implement-gettokensactions

Conversation

@melisaguevara
Copy link
Copy Markdown
Contributor

@melisaguevara melisaguevara commented Aug 21, 2025

This PR adds a new action to fetch tokens supported by the swap API.

@linear
Copy link
Copy Markdown

linear bot commented Aug 21, 2025

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Aug 21, 2025

🦋 Changeset detected

Latest commit: 0230730

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

This PR includes changesets to release 1 package
Name Type
@across-protocol/app-sdk Patch

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

@vercel
Copy link
Copy Markdown

vercel bot commented Aug 21, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
toolkit-example Ready Ready Preview Comment Aug 25, 2025 3:14pm

ashwinrava
ashwinrava previously approved these changes Aug 22, 2025
Copy link
Copy Markdown
Member

@ashwinrava ashwinrava left a comment

Choose a reason for hiding this comment

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

Looks good!

gsteenkamp89
gsteenkamp89 previously approved these changes Aug 22, 2025
Copy link
Copy Markdown
Collaborator

@gsteenkamp89 gsteenkamp89 left a comment

Choose a reason for hiding this comment

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

Nice addition!
Add a changeset for the next release

Copy link
Copy Markdown
Collaborator

@dohaki dohaki left a comment

Choose a reason for hiding this comment

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

I left one nit on naming. Let me know what you think. Also take a look at G's comment and the auto-generated changeset comment here.

dohaki
dohaki previously approved these changes Aug 25, 2025
@melisaguevara melisaguevara changed the title feat: add getSwapApiTokens action feat: add getSwapTokens action Aug 25, 2025
Copy link
Copy Markdown
Collaborator

@gsteenkamp89 gsteenkamp89 left a comment

Choose a reason for hiding this comment

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

TY!

@melisaguevara melisaguevara merged commit 4d33487 into master Aug 25, 2025
3 checks passed
@melisaguevara melisaguevara deleted the mguevara/acx-4320-implement-gettokensactions branch August 25, 2025 15:32
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.

4 participants