Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: enable testnet OTE flows #446

Merged
merged 5 commits into from
Jun 19, 2024
Merged

Conversation

yogurtandjam
Copy link
Contributor

@yogurtandjam yogurtandjam commented Jun 17, 2024

NOTE: This doesn't fully enable testnet. We cannot perform withdrawals and deposits. But this is still past parity with existing squid integration as the route endpoint always fails on testnets.

Squid integration: fails on /v1/route. Withdraw / Deposit button is never enabled, fees never estimated, just get a route error.
Skip Integration (this PR): succeeds on /msgs_direct, button is enabled, fees estimated. Failure upon clicking the button.

@yogurtandjam yogurtandjam changed the title feat: enable testnet OTE feat: top level withdrawal methods [OTE-389] [4/n] Jun 17, 2024
Copy link

linear bot commented Jun 17, 2024

@yogurtandjam yogurtandjam changed the title feat: top level withdrawal methods [OTE-389] [4/n] feat: enable testnet OTE flows Jun 17, 2024
feat: enable withdrawals. also small fixes

Bump version

cleanup suppression

remove squid integrator id check from skip method

address detekt complaint

cleanup
@yogurtandjam yogurtandjam merged commit 796e39e into jerms/QA-route-2 Jun 19, 2024
2 of 3 checks passed
@yogurtandjam yogurtandjam deleted the jerms/QA-route-3 branch June 19, 2024 21:02
yogurtandjam added a commit that referenced this pull request Jun 19, 2024
yogurtandjam added a commit that referenced this pull request Jun 19, 2024
yogurtandjam added a commit that referenced this pull request Jun 21, 2024
yogurtandjam added a commit that referenced this pull request Jun 21, 2024
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