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

Move vesting accunt out of x/auth #10605

Closed
1 of 4 tasks
robert-zaremba opened this issue Nov 24, 2021 · 3 comments
Closed
1 of 4 tasks

Move vesting accunt out of x/auth #10605

robert-zaremba opened this issue Nov 24, 2021 · 3 comments
Labels
C:x/auth S:needs architecture Needs a architecture proposal for how the feature will be implemented

Comments

@robert-zaremba
Copy link
Collaborator

Summary

As we know, x/auth is overloaded. It's primary functionality should be account authentication. But it contains much more.

Problem Definition

Recently we had a problem with x/auth migration where vesting account were vanished: #10591

Proposal

Move vesting account feature from x/auth to x/vesting and enable more feature development in a new module.


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@robert-zaremba robert-zaremba added C:x/auth S:needs architecture Needs a architecture proposal for how the feature will be implemented labels Nov 24, 2021
@robert-zaremba
Copy link
Collaborator Author

cc: @AmauryM , @aaronc

@amaury1093
Copy link
Contributor

duplicate #9958 ? We can close that one though, because this one has a description.

@robert-zaremba
Copy link
Collaborator Author

Thanks, I will close this one and move documentation to the #9958

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:x/auth S:needs architecture Needs a architecture proposal for how the feature will be implemented
Projects
None yet
Development

No branches or pull requests

2 participants