Skip to content

Fix D4A Documentation ommiting that your bot needs to have sufficient Powerlevel to write to the policy list that is its management room. #3232

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

Merged
merged 1 commit into from
Mar 20, 2024

Conversation

FSG-Cat
Copy link
Contributor

@FSG-Cat FSG-Cat commented Mar 19, 2024

In the process of writing the Draupnir for all role documentation it was forgotten that Draupnir needs to have the ability to write to the main management room policy list that controls who can access the bot. This flaw was overlooked during development as naturally without thinking the bot had these powers.

This is fixed upstream in the-draupnir-project/Draupnir#299

Edit: Was opened as draft by mistake.

In the process of writing the Draupnir for all role documentation it was forgotten that Draupnir needs to have the ability to write to the main management room policy list that controls who can access the bot. This flaw was overlooked during development as naturally without thinking the bot had these powers.

Upstream Docs had this exact bug also and the author of this commit will have to go and fix upstream docs also to resolve this bug.
@FSG-Cat FSG-Cat marked this pull request as ready for review March 19, 2024 21:05
@spantaleev spantaleev merged commit a000386 into spantaleev:master Mar 20, 2024
Copy link

github-actions bot commented Apr 7, 2025

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants