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

cw20 hooks and DAO transfer rules #806

Draft
wants to merge 1 commit into
base: development
Choose a base branch
from
Draft

Conversation

JakeHartnell
Copy link
Member

  • Pulls cw20 into the repo (with commit history).
  • Adds transfer hooks to cw20
  • New contract that allows for only transfering tokens between DAO members

Copy link

codecov bot commented Mar 25, 2024

Codecov Report

Attention: Patch coverage is 92.88527% with 271 lines in your changes are missing coverage. Please review.

Project coverage is 96.07%. Comparing base (3ab7017) to head (4737691).

❗ Current head 4737691 differs from pull request most recent head ce8cb3a. Consider uploading reports for the commit ce8cb3a to get more accurate results

Files Patch % Lines
packages/cw20/src/helpers.rs 0.00% 70 Missing ⚠️
contracts/external/cw20-token/src/contract.rs 96.96% 59 Missing ⚠️
...s/external/dao-cw20-transfer-rules/src/contract.rs 59.70% 54 Missing ⚠️
packages/cw20/src/denom.rs 0.00% 21 Missing ⚠️
packages/cw20/src/bin/schema.rs 0.00% 18 Missing ⚠️
packages/cw20/src/balance.rs 75.00% 13 Missing ⚠️
contracts/external/cw20-token/src/msg.rs 88.88% 8 Missing ⚠️
contracts/external/cw20-token/src/bin/schema.rs 0.00% 7 Missing ⚠️
packages/cw20/src/coin.rs 50.00% 7 Missing ⚠️
packages/cw20/src/query.rs 58.33% 5 Missing ⚠️
... and 6 more
Additional details and impacted files
@@               Coverage Diff               @@
##           development     #806      +/-   ##
===============================================
- Coverage        96.26%   96.07%   -0.19%     
===============================================
  Files              209      229      +20     
  Lines            53514    57299    +3785     
===============================================
+ Hits             51513    55051    +3538     
- Misses            2001     2248     +247     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@JakeHartnell JakeHartnell force-pushed the dao-cw20 branch 3 times, most recently from 7cff502 to 4737691 Compare March 26, 2024 13:10
@NoahSaso NoahSaso force-pushed the dao-cw20 branch 3 times, most recently from 35ef237 to d943eb4 Compare April 16, 2024 22:15
@NoahSaso NoahSaso changed the title Dao cw20 cw20 hooks and DAO transfer rules Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants