safe-global / safe-apps-list

MIT License
20 stars 31 forks source link

Add KyberSwap dApp #391

Open tienkane opened 2 months ago

tienkane commented 2 months ago

Entry type

App info

URL: https://kyberswap.com/

Name: KyberSwap

Description: KyberSwap is a multi-chain aggregator and DeFi hub that empowers users with the insights and tools to achieve financial autonomy. All the above while being fast, secure, and easy-to-use.

Icon (PNG, 180x180): KyberSwap

Homepage: https://kyberswap.com/ Twitter: https://x.com/kybernetwork GitHub: https://github.com/KyberNetwork Discord: https://discord.gg/kyberswap

App supports batching multiple transactions via Safe: yes

Supported networks

- Ethereum
- Arbitrum
- Optimism
- zkSync Era
- Polygon PoS
- Base
- BNB Chain
- Linea
- Mantle
- Scroll
- Avalanche
- Fantom
- Blast
- X Layer
- Polygon zkEVM
- Cronos
- BitTorrent

Revision checks

Audit document

https://docs.kyberswap.com/security/audits

Code for review

https://github.com/KyberNetwork/kyberswap-interface

Team information

Company: Kyber Network

Official website: https://kyber.network/

Point of contact: Hugo

Email/Telegram: hugo@kyber.network

kirkkonen commented 2 months ago

This submission was checked and approved by the Product!

francovenica commented 1 month ago

Tried to add this app as a custom one but I get an error. Another URL of the app that I could try?

I tried "https://kyberswap.com/swap/zksync", "https://kyberswap.com/swap/", "https://kyberswap.com/" and none of those work. Tried in other networks as well.

image

tienkane commented 1 month ago

Hi @francovenica sorry for the late reply, I have fixed it, please check again with https://kyberswap.com/

liliya-soroka commented 2 weeks ago

Hi @tienkane , thank you for the fix. We have one comment related to the cross-chain swap: As the safe can not be replayed on all networks, we don't allow the cross-chain swap without the possibility of adding a different recipient on another network. Please disable the cross-chain swap for the safe app or add a recipient field.

image
tienkane commented 2 weeks ago

Hi @liliya-soroka, thank you for your feedback. I will consult with the Product team to assess the feasibility of your suggestion and provide a detailed response as soon as possible.

tienkane commented 1 week ago

Hi @liliya-soroka, We've already hidden the cross-chain swap section for the safe app. Please check. Thank you!

liliya-soroka commented 3 days ago

The state of the tx in n/m safes ( n>1) is not updated even after execution on-chain. As a result , the users are blocked and can not continue swap after the approve Image

cc @katspaugh

katspaugh commented 2 days ago

@tienkane the Safe Apps SDK/provider returns a safeTxHash when submitting a Safe transaction. You can use this hash to get the transaction's status.

It's different from regular wallets where you get an actual tx hash immediately, since Safe txs first go to our queue and have to be signed and subsequently executed.

tienkane commented 1 day ago

@liliya-soroka @katspaugh Thank you for your support in addressing this issue. I will implement the necessary changes and keep you updated on the progress