Skip to content

Conversation

tommasini
Copy link
Contributor

@tommasini tommasini commented Mar 15, 2024

This api-spec is a following work of the MIP of wallet_swapAsset: MetaMask/metamask-improvement-proposals#25

@tommasini tommasini requested a review from a team as a code owner March 15, 2024 20:17
tommasini added 2 commits May 20, 2024 16:22
…erty to optional and added to the description that defaults to 0
adonesky1
adonesky1 previously approved these changes Jun 10, 2024
Copy link
Contributor

@adonesky1 adonesky1 left a comment

Choose a reason for hiding this comment

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

LGTM!

tommasini and others added 2 commits November 14, 2024 19:50
Co-authored-by: Alex Donesky <adonesky@gmail.com>
Co-authored-by: Alex Donesky <adonesky@gmail.com>
@tommasini tommasini merged commit 7296916 into main Nov 14, 2024
7 checks passed
@tommasini tommasini deleted the wallet_swapAsset branch November 14, 2024 20:06
alexandratran added a commit that referenced this pull request Feb 26, 2025
alexandratran added a commit that referenced this pull request Mar 1, 2025
alexandratran pushed a commit to alexandratran/api-specs that referenced this pull request Mar 1, 2025
* wallet_swapAsset v1

* add patterns to the user address and tokens address

* update error codes and remove referral code, also edit the value property to optional and added to the description that defaults to 0

* remove bad identation

* update api spec with the new params accordingly and remove sendTo optional param

* add schema for caip10 addresses and update address format on swap asset rpc method

* update error code

* added more error to the api spec regarding caip 10 addresses

* the swap could not be completed as requested

* add schema for Caip10Address

* remove invalid params error from the api spec

* change the code of swap not available on chain to 36000

* Update openrpc.yaml

Co-authored-by: Alex Donesky <adonesky@gmail.com>

* Update openrpc.yaml

Co-authored-by: Alex Donesky <adonesky@gmail.com>

---------

Co-authored-by: Alex Donesky <adonesky@gmail.com>
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