-
Notifications
You must be signed in to change notification settings - Fork 45
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
feat: cellarfees v2 changes #289
Merged
+5,110
−1,334
Merged
Changes from 1 commit
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
5ffb9f1
WIP - cellarfees v2 protos
cbrit a3611f6
FeeTokenBalance(s) queries and helper functions
cbrit d8cb5a5
Remove previous upgrade wiring
cbrit 22b5272
Remove gravity keeper and hooks from cellarfees
cbrit b1b6f13
Fix imports
cbrit 348c5b5
Fix expected keeper
cbrit 14fd4e7
remove empty file
cbrit 6970c70
Initial update to auction handling
cbrit 80fce3e
Wire up new param
cbrit 4ac6cc4
Remove old upgrade files from build
cbrit f8a6ee1
Refactor cellarfees BeginBlocker
cbrit 4de900c
Update cellarfees keeper unit tests
cbrit a62b45a
Remove empty file
cbrit a42def9
Fix broken unit tests and compiler errors
cbrit 2abf19f
gofmt
cbrit d49043a
WIP - fix integration tests
cbrit 2c5aade
WIP - Integration test updates
cbrit dc341be
Remove auction requirement that denom be prefixed with gravity
cbrit 53dc56c
WIP - Integration test passing, fee acct usomm balance not going to 0…
cbrit 8f8aeb3
Fix test and skip usomm in auction start logic
cbrit e92366c
Linting
cbrit f60fb32
Remove old upgrades from linting rules
cbrit 8ea06d3
Add CLI commands for new queries
cbrit f9111c8
Fix bugs found in testing
cbrit f157ed0
Refactor FeeBalance methods/handler to iterate balances instead of to…
cbrit 9f5ab51
Tweak comments and imports
cbrit a723644
Refactor beginAuction method
cbrit bff3e20
Comment in proto
cbrit File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Remove previous upgrade wiring
commit d8cb5a5ffa8b64c227de2e70dd36bb5baf283d02
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The use of
math/rand
for generating random values in blockchain or security-sensitive contexts is not recommended due to its predictability. It's advisable to usecrypto/rand
instead to ensure cryptographic security.