Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

backport: #8865 vesting account fix #9113

Closed
RoHoss opened this issue Apr 14, 2021 · 7 comments
Closed

backport: #8865 vesting account fix #9113

RoHoss opened this issue Apr 14, 2021 · 7 comments

Comments

@RoHoss
Copy link

RoHoss commented Apr 14, 2021

It is not fixed.

grafik

grafik

I need a solution for this issue. Who can help?

Originally posted by @RoHoss in #9013 (comment)

@RoHoss
Copy link
Author

RoHoss commented Apr 14, 2021

"@type": "/cosmos.vesting.v1beta1.DelayedVestingAccount",
"base_vesting_account": {
"base_account": {
"address": "cro1xxxxxxxxxxxxx",
"pub_key": null,
"account_number": "0",
"sequence": "0"
},
"original_vesting": [
{
"denom": "basecro",
"amount": "632232016211"
}
],
"delegated_free": [],
"delegated_vesting": [],
"end_time": "1648170000"

Is delegating / send with this account possible or first after the end_time?

@tac0turtle
Copy link
Member

Hey, thanks for opening this issue. We do have a bug template that we recommend following.

What version of the software is your node running? Has it been updated?

@tac0turtle
Copy link
Member

The fix has not been backported it seems, #8865. I couldn't find it. this is why you have not had the issue fixed.

@tac0turtle tac0turtle changed the title It is not fixed. backport: #8865 vesting account fix Apr 14, 2021
@aaronc
Copy link
Member

aaronc commented Apr 14, 2021

This can't be backported because it is state breaking and requires a migration for live chains. New zones can fork the SDK temporarily (like we did), but most zones will need the migration which will be included in 0.43.

@tac0turtle
Copy link
Member

oh okay. Thank you for the explanation!

@RoHoss You will have to wait till the next upgrade. The funds are safe.

@aaronc
Copy link
Member

aaronc commented Apr 14, 2021

If crypto.com wants to address sooner they can. I believe Akash did something similar. The fact there is an unsupported type error makes me think there may be some other issue. If crypto.com maintainers want support in figuring out an intermediate solution they can let us know, but there's no easy immediate fix in the SDK.

@RoHoss
Copy link
Author

RoHoss commented Apr 14, 2021

Thanks for info. I transport it to CDC.

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

No branches or pull requests

3 participants