Skip to content

fix: Remove astroport from skip client#2100

Merged
dankim214 merged 3 commits into
mainfrom
dkim/skip-exclusion
Mar 23, 2026
Merged

fix: Remove astroport from skip client#2100
dankim214 merged 3 commits into
mainfrom
dkim/skip-exclusion

Conversation

@dankim214
Copy link
Copy Markdown
Collaborator

neutron-astroport was being very slow

@dankim214 dankim214 requested a review from a team as a code owner March 23, 2026 14:24
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 23, 2026

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

Project Deployment Actions Updated (UTC)
v4-staging Ready Ready Preview, Comment Mar 23, 2026 7:01pm
v4-testnet Ready Ready Preview, Comment Mar 23, 2026 7:01pm

Request Review

Comment thread src/constants/skip.ts Outdated

// neutron-astroport excluded due to degraded performance causing slow route simulations
export const SKIP_SWAP_VENUES = [
{ chainId: 'osmosis-1', name: 'osmosis-poolmanager' },
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Osmosis as well?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

OH this isn't an exclusion

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

let me ask if neutron duality is ok

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

yeah theres no exclusion field :(

@dankim214 dankim214 enabled auto-merge (squash) March 23, 2026 18:54
@dankim214 dankim214 merged commit efb9555 into main Mar 23, 2026
13 checks passed
@dankim214 dankim214 deleted the dkim/skip-exclusion branch March 23, 2026 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants