-
Notifications
You must be signed in to change notification settings - Fork 20
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
Mark guild Role position as computed #171
Mark guild Role position as computed #171
Conversation
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.
thx for your PR
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.
I think some people may want to set the roll position manually.
This PR is too destructive, isn’t it?
You are still able to set the position of the role. By swapping to from Default to Computed, roles that have their position not set and are updated remotely won't trigger changes. If you have a value set for the position attribute, then changes will still be triggered. |
I see. Thanks! |
Mark the position of a role as computed. I find it is easier to create roles with terraform and then move them into position manually.
Currently, I have to use
and this would alleviate that.
Shouldn't be a breaking change but also might be worth releasing as a 2.0