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.

Sign up for GitHub

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

Jump to bottom

FeeTx.FeePayer and FeeTx.FeeGranter should be of the same type #16183

Closed
Tracked by #15677
kocubinski opened this issue May 16, 2023 · 3 comments · Fixed by #16272
Closed
Tracked by #15677

FeeTx.FeePayer and FeeTx.FeeGranter should be of the same type #16183

kocubinski opened this issue May 16, 2023 · 3 comments · Fixed by #16272
Labels
T:feature-request

Comments

@kocubinski
Copy link
Member

Ref: #15284 (comment)

Interface methods were refactored from both as AccAddress to one as []byte and one as string. We should pick just one, probably []byte.
@aaronc aaronc mentioned this issue May 16, 2023
Closed
8 tasks
@vishal-kanna
Copy link
Contributor

vishal-kanna commented May 22, 2023 •
edited
Loading

@kocubinski is any one working on this issue ,if not i would like to work on this issue.

@tac0turtle
Copy link
Member

yea feel free to pick it up, let us know if you need any help

@vishal-kanna
Copy link
Contributor

@tac0turtle okay then i'll be working on the issue

@vishal-kanna vishal-kanna mentioned this issue May 24, 2023
Merged
19 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
T:feature-request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: changed the data type for FeePayer and FeeGranter vishal-kanna/cosmos-sdk
3 participants
@kocubinski @tac0turtle @vishal-kanna