docs: Update dai_token_migration.adoc with other Swap options

starknet-io / starknet-docs

The repo for Starknet's developer documentation. Includes contribution guidelines and the Starknet documentation supplementary style guide

https://docs.starknet.io/

112 stars 158 forks source link

Closed stoobie closed 2 months ago

stoobie commented 2 months ago

Added more swap options, as per request from Maxime.

[x] Changes have been done against main branch, and PR does not conflict
[x] PR title follows the convention: <docs/feat/fix/chore>(optional scope): <description>, e.g: fix: minor typos in code

This change is

github-actions[bot] commented 2 months ago

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-1237/documentation/ .

stoobie commented 2 months ago

I would switch the order - swapping on chain is more preferable than using StarkGate. Moreover, it might be nicer to have a list with 2 columns.

@natan-granit What would be in the second column?

stoobie commented 2 months ago

@natan-granit PTAL

github-actions[bot] commented 2 months ago

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-1237/documentation/ .