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

[SWA-146][FEAT] - Add Openocean support #1913

Merged
merged 12 commits into from
Mar 22, 2024

Conversation

ElRodrigote
Copy link
Collaborator

@ElRodrigote ElRodrigote commented Mar 15, 2024

Fixes: SWA-146

Description

  • Add Kleros as a routing platform

Visual Evidence

image

How to test the changes

  1. Pull this branch
  2. Run the project locally
  3. Go to Swapr dapp page
  4. Connect your wallet
  5. Go to the Swapbox, select the pair you want to swap on any of the supported chains by Openocean (Arbitrum, BSC, Gnosis, Ethereum, Optimism, Polygon, Scroll) and Openocean should appear as a valid route.

@ElRodrigote ElRodrigote force-pushed the feat/swa-146-add-openocean-support branch from 82aa3ec to 5919ef3 Compare March 15, 2024 19:11
@ElRodrigote ElRodrigote force-pushed the feat/swa-146-add-openocean-support branch from 5919ef3 to 102e45a Compare March 21, 2024 00:30
Copy link

netlify bot commented Mar 21, 2024

Deploy Preview for swapr ready!

Name Link
🔨 Latest commit 73342dc
🔍 Latest deploy log https://app.netlify.com/sites/swapr/deploys/65fdb76a94dcc300081abde8
😎 Deploy Preview https://deploy-preview-1913--swapr.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 25 (no change from production)
Accessibility: 79 (no change from production)
Best Practices: 92 (no change from production)
SEO: 83 (no change from production)
PWA: 80 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

Comment on lines 310 to 318
if (platform.name === RoutablePlatform.OPENOCEAN.name) {
return SwaprV3Trade.getQuote({
quoteCurrency: currencyIn,
amount: currencyAmountOut,
maximumSlippage,
recipient: receiver,
tradeType: TradeType.EXACT_OUTPUT,
})
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

This doesn't look right 😄

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ahhh it was a test and you passed (?
Lol sorry, addressing it right now

Copy link
Collaborator

@berteotti berteotti left a comment

Choose a reason for hiding this comment

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

Good job, looking good!

@berteotti
Copy link
Collaborator

Don't forget to merge this only after updating Swapr SDK version

@ElRodrigote ElRodrigote force-pushed the feat/swa-146-add-openocean-support branch from fa228cf to 73342dc Compare March 22, 2024 16:52
Copy link
Collaborator

@berteotti berteotti left a comment

Choose a reason for hiding this comment

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

Looking good, don't forget to rebase and squash commit before merging

@ElRodrigote ElRodrigote merged commit 3e9860d into develop Mar 22, 2024
16 of 17 checks passed
ElRodrigote added a commit that referenced this pull request Mar 25, 2024
* feat(SWA-146): add Openocean support (#1913)
* docs(SWA-138): update footer links to docs URLs

---------

Co-authored-by: Berteotti <berteotti2@gmail.com>
ElRodrigote added a commit that referenced this pull request Mar 25, 2024
release: v1.0.0-beta.22.3
* feat(SWA-146): add Openocean support (#1913)
* docs(SWA-138): update footer links to docs URLs
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