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

Decouple auth module from other SDK top-level core packages #7318

Closed
4 of 9 tasks
robert-zaremba opened this issue Sep 15, 2020 · 3 comments
Closed
4 of 9 tasks

Decouple auth module from other SDK top-level core packages #7318

robert-zaremba opened this issue Sep 15, 2020 · 3 comments
Labels
C:x/auth Type: Code Hygiene General cleanup and restructuring of code to provide clarity, flexibility, and modularity.

Comments

@robert-zaremba
Copy link
Collaborator

robert-zaremba commented Sep 15, 2020

Summary

Related to

We still some modules doing cross imports.
Example: packages from x/auth are imported in client/tx and in x/bank/.

/cc @alessio


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@robert-zaremba robert-zaremba added the Type: Code Hygiene General cleanup and restructuring of code to provide clarity, flexibility, and modularity. label Sep 15, 2020
@clevinson
Copy link
Contributor

Ideally some of this can be tackled more broadly when looking at #7091

@robert-zaremba
Copy link
Collaborator Author

Ideally some of this can be tackled more broadly when looking at #7091

Thanks, I've mentioned this in the issue OP.

@tac0turtle
Copy link
Member

this is part of app wiring work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:x/auth Type: Code Hygiene General cleanup and restructuring of code to provide clarity, flexibility, and modularity.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants