Skip to content
This repository was archived by the owner on Apr 8, 2026. It is now read-only.

Fix breaking Hub 3 / Gaia 2 changes - #3

Open
jordaaash wants to merge 16 commits into
masterfrom
hub-3-gaia-2
Open

Fix breaking Hub 3 / Gaia 2 changes#3
jordaaash wants to merge 16 commits into
masterfrom
hub-3-gaia-2

Conversation

@jordaaash

Copy link
Copy Markdown
Contributor

No description provided.

@jordaaash

Copy link
Copy Markdown
Contributor Author

Notes to self:

Publish on npm as 0.6.0 + next tag. Add a note to the readme when this is ready for production.

GopherJS currently requires go 1.12, while the latest SDK requires go 1.13. This may be a non-issue as this library doesn't build against the SDK source directly, but something to be aware of.

@hukkin

hukkin commented Dec 9, 2019

Copy link
Copy Markdown

hey @jordansexton dont want to put on any pressure, but do you perhaps know already if this will be merged/released before the cosmoshub-3 fork?

@jordaaash

Copy link
Copy Markdown
Contributor Author

@hukkinj1 Unfortunately it will not completed by then (the upgrade is set to occur in December 11). It is the highest priority thing I'm working on right now though. Will this affect any production applications you are running?

@hukkin

hukkin commented Dec 10, 2019

Copy link
Copy Markdown

It will have an effect on a production application yes, so if you you have e.g. special tips on how to make this run prior to the merge/release, that would be much appreciated (assuming the state of the branch is e.g. such that most transactions decode already?). Otherwise no stress, we'll just have to be patient then :)

@jordaaash

Copy link
Copy Markdown
Contributor Author

I have pushed a new branch to https://github.com/cosmos/amino-js/tree/hub-3-gaia-2-alpha

This has also been published as 0.7.1-alpha.1 on npm under the next dist tag. You should be able to install @tendermint/amino-js@0.7.0-alpha.1 to try it out.

This release is more or less just current the master branch @ 9d133ac with 812cbf4 cherry-picked and tests updated accordingly.

It doesn't include any of the actual module changes in the current version of the SDK or gaia 2.0.3 used in the Hub 3 upgrade, and will likely be missing some of the new types, or have different structures for some of them.

Therefore, it might work for what you're doing, or it might not. I do not recommend relying on this for anything in production, but if you have no alternative, it might help until there is a proper release based on code generation from the SDK.

@hukkin

hukkin commented Dec 12, 2019

Copy link
Copy Markdown

@jordansexton a thousand thanks, you're the best! FYI: the alpha build has successfully decoded every cosmoshub-3 transaction since around 17:30 UTC yesterday.

@romeo4934

Copy link
Copy Markdown

Hi Jordan,
Do you know when your PR will be merged?

@jordaaash

Copy link
Copy Markdown
Contributor Author

I apologize to everyone for the delay on updates here!

There's been a lot of internal discussion and progress in several draft PRs on converting the Cosmos SDK to use Protobuf3 directly and eventually remove or limit the use of Amino. This led me to slow down on development of Amino-based solutions and amino-js while I was participating in these.

At the outset, I thought it might prove unnecessary to complete this if Amino is not used by the SDK. However, it looks like that work will be incremental and take time, so I've resumed work on this PR. I'll have a sense of how much work is left this week and will update accordingly.

For now, the alpha build I recommended @ #3 (comment) seems functional, but I would not trust it for production use. I have reprioritized keeping this library current with the SDK and Tendermint, and will follow up soon.

@romeo4934

Copy link
Copy Markdown

Sounds good! We will release iov-core 2.0 after that :)

@romeo4934

Copy link
Copy Markdown

any progress :) ? @jordansexton

@jordaaash

Copy link
Copy Markdown
Contributor Author

@antoineherzog The work on this is incomplete, but I'm on it! It was a pretty long, tedious process the first time around. I've been attempting a new solution based on reflection over the SDK and Tendermint. I'll be sure to update here and in Telegram when I have something for you.

@romeo4934

Copy link
Copy Markdown

Great :)))

@romeo4934

Copy link
Copy Markdown

Do you have any new ETA Jordan on this PR?

@romeo4934

Copy link
Copy Markdown

Let us know :)

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.

3 participants