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

requireOwner is too restrictive in token modules #2774

Open
dk1a opened this issue Apr 30, 2024 · 0 comments · May be fixed by #2777
Open

requireOwner is too restrictive in token modules #2774

dk1a opened this issue Apr 30, 2024 · 0 comments · May be fixed by #2777

Comments

@dk1a
Copy link
Member

dk1a commented Apr 30, 2024

I think requireAccess would make more sense for token mints/burns, and it allows more than 1 address to perform these operations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
MUD
Status: No status
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(world-modules): use requireAccess for token mint and burn latticexyz/mud
1 participant
@dk1a