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

RandToken burnFromAdmin #64

Open
adradr opened this issue Aug 16, 2023 · 0 comments
Open

RandToken burnFromAdmin #64

adradr opened this issue Aug 16, 2023 · 0 comments
Assignees
Labels
question Further information is requested

Comments

@adradr
Copy link
Collaborator

adradr commented Aug 16, 2023

Summary:

Currentlly, the burnFromAdmin function is implemented in the RandToken contract as:

    function burnFromAdmin(
        address account,
        uint256 amount
    ) public whenNotPaused onlyRole(DEFAULT_ADMIN_ROLE) {
        _burn(account, amount);
    }

Question is what our goals with this? Do we wanna keep the option to burn any token from any user, or we would simply burn our own treasury tokens?

Motivation:

We need to decide what we want to do with this function since the current naming of the function and its capabilities might frighten some users.

@adradr adradr added the question Further information is requested label Aug 16, 2023
@adradr adradr self-assigned this Aug 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant