-
Notifications
You must be signed in to change notification settings - Fork 16
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
Feat: DAO NFT based #657
base: main
Are you sure you want to change the base?
Feat: DAO NFT based #657
Conversation
✅ Deploy Preview for testitori ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for teritori-dapp ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
411c1a7
to
0bb6b09
Compare
a3082c9
to
55b22ce
Compare
200b5ae
to
ab643a7
Compare
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.
did not read all yet
|
||
_ = s.conf.IndexerDB.Raw(` | ||
select distinct owner from ( | ||
select nfts.id, max(a.time) atime, COALESCE(m.buyer_id, t.buyer_id, tn.receiver) as owner |
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.
why aggregate instead of using owner_id
field?
Signed-off-by: Norman Meier <[email protected]>
810da28
to
9673f76
Compare
https://github.com/DA0-DA0/dao-contracts/blob/main/contracts/voting/dao-voting-cw721-staked/src/contract.rs
Use a local indexer for now