Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Implement k.Logger() interface on keeper #3512

Closed
cwgoes opened this issue Feb 6, 2019 · 2 comments · Fixed by #4625
Closed

Implement k.Logger() interface on keeper #3512

cwgoes opened this issue Feb 6, 2019 · 2 comments · Fixed by #4625
Assignees
@aayushijain23
Labels
good first issue

Comments

@cwgoes
Copy link
Contributor

cwgoes commented Feb 6, 2019

k.Logger(ctx) should return ctx.Logger().With("module", ModuleName)
@jackzampolin
Copy link
Member

cc @fedekunze @sabau

@alexanderbez
Copy link
Contributor

cc @aayushijain23

@aayushijain23 aayushijain23 self-assigned this Jun 24, 2019
@aayushijain23 aayushijain23 mentioned this issue Jun 25, 2019
Merged
5 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@aayushijain23 aayushijain23

Labels
good first issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Implement logger interface on keeper. cosmos/cosmos-sdk
5 participants
@alexanderbez @jackzampolin @cwgoes @aayushijain23 @fedekunze