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

Missing inherited roles in generated artifact #1600

Open
macor161 opened this issue Apr 2, 2020 · 1 comment
Open

Missing inherited roles in generated artifact #1600

macor161 opened this issue Apr 2, 2020 · 1 comment
Labels
🐛 bug An unexpected behavior of the API 🦅 high priority A signal from the teams to mark the importance of an issue to the maintainers

Comments

@macor161
Copy link
Contributor

macor161 commented Apr 2, 2020

🐛 Bug Report

Roles inherited from another smart contract are not included in the generated artifact.json file. E.g. using the following command in the Agent app won't generate the TRANSFER_ROLE role:

aragon apm publish --only-artifacts major
@macor161 macor161 self-assigned this Apr 2, 2020
@macor161 macor161 added 🐛 bug An unexpected behavior of the API 🦅 high priority A signal from the teams to mark the importance of an issue to the maintainers labels Apr 2, 2020
@sohkai
Copy link
Contributor

sohkai commented Apr 2, 2020

It should be noted that it is not necessarily the inherited roles—there's no way inherited roles would appear at the moment.

The current problem with the Agent is that its TRANSFER_ROLE is declared in its arapp.json, but it is not being applied correctly to the resulting artifact.json.

@eliobricenov eliobricenov self-assigned this Mar 1, 2021
@eliobricenov eliobricenov removed their assignment Aug 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🐛 bug An unexpected behavior of the API 🦅 high priority A signal from the teams to mark the importance of an issue to the maintainers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@macor161 @sohkai @eliobricenov