-
Notifications
You must be signed in to change notification settings - Fork 6
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
Flat rate whitelist support [SG-280] #253
Flat rate whitelist support [SG-280] #253
Conversation
this |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## tasiov/sg-1051-names-renewal #253 +/- ##
================================================================
- Coverage 70.68% 70.51% -0.18%
================================================================
Files 26 30 +4
Lines 1699 1930 +231
================================================================
+ Hits 1201 1361 +160
- Misses 498 569 +71 ☔ View full report in Codecov by Sentry. |
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.
It looks like this PR includes a version update to CW that changes code in many of the contracts along with the new whitelist. Can they broken into two PRs to make it easier to review? Maybe the CW update first?
I've updated this PR to revert the updating of the deprecated functions which is lighting up the linter. I'm not sure why its failing the linter for this while the base branch isn't. |
* minor updates * ignore lint * ignore lint * update schema
let list = whitelists.iter().find(|whitelist| match whitelist { | ||
WhitelistContract::Updatable(updatable) => updatable | ||
.includes(&deps.querier, sender.to_string()) | ||
.unwrap_or(false), | ||
WhitelistContract::Flatrate(flatrate) => flatrate | ||
.includes(&deps.querier, sender.to_string()) | ||
.unwrap_or(false) | ||
.unwrap_or(false), | ||
}); |
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.
Remember receiving Generic error: Querier contract error: codespace: wasm, code: 9: execute wasm contract failed
errors during on-chain tests in these match statements. Will need to test these on chain again.
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.
@MightOfOaks I've merged in the changes from the dYDx-airdrop branch. im curious if we can figure out why this is not matching correctly
8c53bdd
into
tasiov/sg-1051-names-renewal
The intent of these changes are to add a flat-rate whitelist to Names. The scope of changes include
updatable-whitelist-flatrate
contract (largely a modified version of the original whitelist code)whitelists
array inname-minter