-
Notifications
You must be signed in to change notification settings - Fork 142
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
Add missing query to dao-pre-propose-approver #766
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## development #766 +/- ##
==============================================
Coverage ? 96.26%
==============================================
Files ? 203
Lines ? 50093
Branches ? 0
==============================================
Hits ? 48221
Misses ? 1872
Partials ? 0 ☔ 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.
just a few comments. not 100% sure about renaming. what do you think about returning an Option<u64>
instead?
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.
Nice! Agree with the comment above about may_load
for the query, but otherwise LGTM.
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.
lets go!
The
dao-pre-propose-approver
contract does not currently provide any query to determine which pre-propose ID corresponds with a given proposal. In other words, you cannot check which pending proposal an approval proposal was made for.This PR adds a query
PendingProposalIdForApprovalProposalId
which takes theid
of the approval proposal being voted on by the approval DAO and returns theid
of the pending proposal in thedao-pre-propose-single
contract on the DAO waiting for proposal approval.