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

Fix voting module native token proposal deposit on instantiate #786

Merged
merged 3 commits into from
Dec 19, 2023

Conversation

NoahSaso
Copy link
Member

Resolves #765

This allows setting up a proposal deposit using the newly created voting module token on DAO instantiation.

@NoahSaso NoahSaso force-pushed the noah/fix-native-voting-module-prop-deposit-init branch from fec882d to 1474cd7 Compare December 17, 2023 22:05
Copy link

codecov bot commented Dec 17, 2023

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

❗ No coverage uploaded for pull request base (development@a3f15d7). Click here to learn what that means.

Files Patch % Lines
...roposal/dao-proposal-multiple/src/testing/tests.rs 99.17% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff               @@
##             development     #786   +/-   ##
==============================================
  Coverage               ?   96.69%           
==============================================
  Files                  ?      204           
  Lines                  ?    52651           
  Branches               ?        0           
==============================================
  Hits                   ?    50913           
  Misses                 ?     1738           
  Partials               ?        0           

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

@NoahSaso NoahSaso force-pushed the noah/fix-native-voting-module-prop-deposit-init branch 2 times, most recently from 49a6916 to d18b04d Compare December 18, 2023 17:39
@NoahSaso NoahSaso force-pushed the noah/fix-native-voting-module-prop-deposit-init branch from d18b04d to 537c2c6 Compare December 18, 2023 17:53
Copy link
Member

@JakeHartnell JakeHartnell left a comment

Choose a reason for hiding this comment

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

LGTM! Wish we had caught this during the first audit.

Made a small note, but don't feel too strongly about it.

@NoahSaso NoahSaso force-pushed the noah/fix-native-voting-module-prop-deposit-init branch from e063c47 to 9a50f20 Compare December 19, 2023 01:35
@NoahSaso NoahSaso merged commit c37349c into development Dec 19, 2023
@NoahSaso NoahSaso deleted the noah/fix-native-voting-module-prop-deposit-init branch December 19, 2023 02:00
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.

Cannot setup proposal deposit on token DAO creation when creating a new Token Factory
2 participants