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

[WIP] Upokt fees #804

Closed
wants to merge 3 commits into from
Closed

[WIP] Upokt fees #804

wants to merge 3 commits into from

Conversation

Olshansk
Copy link
Member

@Olshansk Olshansk commented Sep 9, 2024 •
edited
Loading

Summary

  • Add tx fees everywhere that bank send comes up.

Issue

Related issues & PRs:

Type of change

Select one or more from the following:

@okdas
Copy link
Member

okdas commented Sep 9, 2024

Let's temporarily turn them off on LocalNet instead: #805
We are going to properly do that in #794

@Olshansk Olshansk requested a review from okdas September 9, 2024 18:56
@Olshansk Olshansk self-assigned this Sep 9, 2024
@Olshansk Olshansk added protocol General core protocol related changes on-chain On-chain business logic labels Sep 9, 2024
@Olshansk Olshansk mentioned this pull request Sep 9, 2024
Merged
@Olshansk
Copy link
Member Author

Olshansk commented Sep 9, 2024

See this thread: #805 (comment)

@Olshansk Olshansk closed this Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@okdas okdas Awaiting requested review from okdas

Assignees

@Olshansk Olshansk

Labels
on-chain On-chain business logic protocol General core protocol related changes
Projects
Shannon
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Olshansk @okdas