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.

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

chore: add atom to default registry #1371

Merged
merged 19 commits into from
Sep 18, 2022
Merged

chore: add atom to default registry #1371

merged 19 commits into from
Sep 18, 2022

Conversation

adamewozniak
Copy link
Collaborator

Add atom's IBC denom to the default whitelist


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

  • included the correct type prefix in the PR title
  • added ! to the type prefix if API or client breaking change
  • added appropriate labels to the PR
  • targeted the correct branch (see PR Targeting)
  • provided a link to the relevant issue or specification
  • added a changelog entry to CHANGELOG.md
  • included comments for documenting Go code
  • updated the relevant documentation or specification
  • reviewed "Files changed" and left comments if necessary
  • confirmed all CI checks have passed

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.

I have...

  • confirmed the correct type prefix in the PR title
  • confirmed all author checklist items have been addressed
  • reviewed state machine logic
  • reviewed API design and naming
  • reviewed documentation is accurate
  • reviewed tests and test coverage
  • manually tested (if applicable)

@adamewozniak adamewozniak changed the title chore: add atom to default oracle params chore: add atom to default registry Sep 12, 2022
@codecov-commenter
Copy link

codecov-commenter commented Sep 12, 2022

Codecov Report

Merging #1371 (7a5d322) into main (d83990a) will increase coverage by 0.44%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1371      +/-   ##
==========================================
+ Coverage   52.52%   52.96%   +0.44%     
==========================================
  Files          70       70              
  Lines        6692     6751      +59     
==========================================
+ Hits         3515     3576      +61     
+ Misses       2909     2907       -2     
  Partials      268      268              
Impacted Files Coverage Δ
x/leverage/types/params.go 21.92% <ø> (ø)
x/oracle/types/params.go 100.00% <ø> (ø)
x/leverage/client/tests/suite.go 95.38% <100.00%> (+0.07%) ⬆️
x/leverage/types/genesis.go 10.34% <100.00%> (+1.14%) ⬆️
x/leverage/types/token.go 83.22% <100.00%> (+10.69%) ⬆️
x/leverage/keeper/genesis.go 4.00% <0.00%> (+1.00%) ⬆️

x/leverage/types/token.go Outdated Show resolved Hide resolved
@toteki toteki marked this pull request as ready for review September 18, 2022 19:53
@toteki toteki requested review from a team as code owners September 18, 2022 19:53
MinCollateralLiquidity: sdk.MustNewDecFromStr("0"),
MaxSupply: sdk.NewInt(100000000000),
},
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need to Atom as well.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't the default registry, it's one that allows us to run CLI tests with UMEE only

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know, but don't we need it for tests?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Switching to default registry here would cause a few downstream tests to fail (they're expecting these specific values) but we can do that at a later date. Created #1414

Copy link
Member

@robert-zaremba robert-zaremba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pre-approving. let's double check about umee.max_collateral_share.

@toteki toteki merged commit b42395a into main Sep 18, 2022
@toteki toteki deleted the woz/atom-default-denom branch September 18, 2022 23:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants