Stakooler refactor #65
Merged
Conversation
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
Collaborator
mircea-c
commented
Oct 1, 2024
mircea-c
commented
- accounts and chains are specified in a json file instead of toml to save on space
- accounts specified in base64 encoding instead of bech32 to avoid duplicate entries in config file
- query chain for prefix instead of parsing it from address
- query asset list from cosmos directory for each chain
- denom metadata is sourced from asset list with fallback to bank module or fail if it cannot be obtained
- fix a number of bugs caused by missing endpoints and differences between sdk versions
- refactor reading in files - update golang version to latest
- fix address decoding - update auth and bank queries - update table display
- fixed csv display - updated distribution queries - fixed double entry bug
- fixed staking details bug
- cleaned up api response processing - fixed some initialization runtime errors
- fixed issue with chains that don't have prefix query - cleaned up table display column names - cleaned up chain registry queries
- fix injective denom query - log account not found errors, without skipping any other accounts - put in an exception for medibloc prefix query returning the wrong value
- tidy up data structures - move api response structs into the api folder - standardize method usage for http query responses
This was referenced Oct 2, 2024
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.