-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
x/auth & x/bank Improvements #7091
Labels
Milestone
Comments
This was referenced Aug 18, 2020
Closed
9 tasks
do-not-edit-start-codetree-epic-issuesIssues in this epic:
do-not-edit-end-codetree-epic-issuesUpdated OP by mentioning multisig and dependencies on x/auth cleanup. |
This was referenced Jul 13, 2021
Closed
4 tasks
every point here has an issue in the repo, closing this |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Summary
While working with the SDK, I have noticed several areas where the
x/auth
andx/bank
modules could be improved. Some good progress was made via ADR 004 which stores balances by an account/denom key pair and via the merging ofx/bank
andx/supply
. This meta-issue tracks several additional proposed improvements.Each potential area for improvement will receive a short description in this issue and then have a larger linked issue which describes the issue and proposed improvements in more detail. Some of these changes will likely require ADRs.
WG Meeting Notes
https://hackmd.io/ORbi5JWgQDeNBdBYoc5H_Q
Current Roadmap (13.07.2021)
Metadata
Details
Remove Bech32 global prefixes
See #9690.
Index supply by denomSee #7092.✅ Done.x/bank
object capabilitiesSee #7093.
Decimal coins
See #7113
Separate vesting from
x/auth
accountsTODO Some initial discussion occurred in #4287.
MultiSig and ed25519
Multisig Schnoor / ed25519 standarization: #7315
Cosmos Scalars
see #9694.
cleanup dependencies on x/auth
Other modules depend on x/auth. This is against Cosmos SDK design principles. Either we need to revise the principles, or do refactoring. NOTE: part of this dependencies should / will be fixed when removing deprecated functionality.
More: #7318
The text was updated successfully, but these errors were encountered: