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.

Sign up for GitHub

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

Jump to bottom

Use cw-ownable everywhere #721

Open
JakeHartnell opened this issue Jul 9, 2023 · 1 comment
Open

Use cw-ownable everywhere #721

JakeHartnell opened this issue Jul 9, 2023 · 1 comment
Labels
tech-debt
Milestone
v3.0

Comments

@JakeHartnell
Copy link
Member

Trying to make APIs for contracts more standardized and consistent between all DAO DAO contracts.

cw-ownable is amazing, and we should just use it everywhere. dao-dao-core currently uses our old Admin pattern.
@JakeHartnell JakeHartnell added tech-debt good-first-issue Issue that is good for first. 2.5 Targeted for v2.5.0 release labels Jul 9, 2023
@JakeHartnell JakeHartnell removed the 2.5 Targeted for v2.5.0 release label Aug 18, 2023
JakeHartnell pushed a commit that referenced this issue Aug 19, 2023
Clean up won't fix TODOs
41e47b5 
Can handle these a separate PR, see #721
@JakeHartnell JakeHartnell added this to the v3.0 milestone Aug 30, 2023
@JakeHartnell
Copy link
Member Author

For those wishing to take on this ticket, I recommend taking it one contract at a time. 😉

@JakeHartnell JakeHartnell removed the good-first-issue Issue that is good for first. label Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
tech-debt
Projects
None yet
Milestone
v3.0
Development

No branches or pull requests
1 participant
@JakeHartnell