Skip to content

Conversation

@Drakeoon
Copy link
Contributor

@Drakeoon Drakeoon commented Nov 7, 2025

Done:

  • Improve the way we display, filter and sort tokens when selecting token
  • Add UI controls for changing /quote and /swap endpoints, allow overriding their API key
  • Fix type issues with the wip branch

Important

This branch was originally created from #1532, but it looks like the commits are made on a fresh branch that was synced with develop. Need to ensure we're using the right SDK approach for checking /swap and /quote endpoints

@Drakeoon Drakeoon changed the base branch from chore/keystore-signing-api-provided-tx to develop November 7, 2025 17:37
@Drakeoon Drakeoon changed the title feat: add ability to configure swap quote v3 endpoints WIP feat: add ability to configure swap quote v3 endpoints Nov 7, 2025
@Drakeoon Drakeoon changed the title WIP feat: add ability to configure swap quote v3 endpoints feat: add ability to configure swap quote v3 endpoints Nov 10, 2025
@Drakeoon Drakeoon requested review from ice-chillios and towanTG and removed request for towanTG November 10, 2025 12:40
const plugin = availablePlugins[spenderAddress];

if (plugin) {
if (type === ApproveMode.CheckOnly && "isAssetValueApproved" in plugin) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

might want to use match here to reduce if structures

@ice-chillios ice-chillios changed the base branch from develop to feat/adds-tx-signing November 13, 2025 12:15
@ice-chillios ice-chillios changed the base branch from feat/adds-tx-signing to develop November 13, 2025 12:15
@ice-chillios ice-chillios force-pushed the develop branch 2 times, most recently from 6735d9c to 94e558a Compare November 13, 2025 12:44
@Drakeoon Drakeoon closed this Nov 13, 2025
@Drakeoon
Copy link
Contributor Author

Closing this PR, I'll split this it into smaller ones:

  1. the UI improvements for token selection
  2. the playground controls & v3 /quote /swap flow logic

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