Skip to content
This repository has been archived by the owner on Oct 13, 2023. It is now read-only.

feat: convertCoin and convertERC20 messages #45

Merged
merged 6 commits into from
Jun 14, 2022
Merged

feat: convertCoin and convertERC20 messages #45

merged 6 commits into from
Jun 14, 2022

Conversation

hanchon
Copy link
Contributor

@hanchon hanchon commented Jun 13, 2022

No description provided.

@hanchon
Copy link
Contributor Author

hanchon commented Jun 13, 2022

Currently testing the amino patch

utils/index.ts Outdated Show resolved Hide resolved
@lgtm-com
Copy link

lgtm-com bot commented Jun 13, 2022

This pull request introduces 3 alerts when merging 4e5c395 into 80bdf98 - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

hanchon and others added 2 commits June 13, 2022 16:52
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
@lgtm-com
Copy link

lgtm-com bot commented Jun 13, 2022

This pull request introduces 3 alerts when merging 27608c5 into 80bdf98 - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Jun 13, 2022

This pull request introduces 3 alerts when merging 8fc65af into 80bdf98 - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

@hanchon
Copy link
Contributor Author

hanchon commented Jun 13, 2022

The LGTM alerts were fixed, they were there because I accidentally pushed the index.ts changes

@hanchon hanchon disabled auto-merge June 14, 2022 09:50
@hanchon hanchon merged commit 9b78984 into main Jun 14, 2022
@hanchon hanchon deleted the erc20Messages branch June 14, 2022 09:50
tiendn pushed a commit to AstraProtocol/astrajs that referenced this pull request Jun 15, 2022
* feat: kyve address converter (evmos#39)

* feat: withdrawValidatorCommission message (evmos#42)

* feat: (wip)withdrawValidatorCommission message

* fix: send the correct params

* ci: publish

 - @tharsis/address-converter@0.1.8
 - @tharsis/eip712@0.2.2
 - evmosjs@0.2.6
 - @tharsis/proto@0.1.17
 - @tharsis/transactions@0.2.3

* feat: utils to sign messages locally (evmos#44)

* feat: utils to sign messages locally

* chore: remove old comments

* feat: convertCoin and convertERC20 messages (evmos#45)

* feat: convertCoin and convertERC20 messages

* Update packages/eip712/src/messages/erc20/msgConvertERC20.ts

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* Update packages/eip712/src/messages/erc20/msgConvertCoin.ts

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* fix: small changes

* chore: revert index.ts util, update localnet example

* chore: remove console log

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* fix: bump version and missing

Co-authored-by: John Letey <john@kyve.network>
Co-authored-by: Guillermo Paoletti <guillermo.paoletti@gmail.com>
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants