Permissions guide #7497
Permissions guide #7497
Conversation
AbstractUmbra
force-pushed
the
docs/permissions-guide
branch
from
dff5225 to
85ac0d1
Compare
docs/guide/topics/permissions.rst
Outdated
|
|
||
| .. code-block:: python3 | ||
|
|
||
| # I have found that the easiest way to do this, is to get the current overwrites for the Role or Member we're editing: |
There was a problem hiding this comment.
Is it the easiest way? This is kind of a rhetorical question, but I feel like we should be confident about what we're putting into a guide. After all, the reader is trusting that the information we've taken time to accumulate is our best-in-class "seal of approval" technique.
We can deliberate as much as we want while writing it, and leave only the information we know is tried and true in the document. There's no reason to clue the reader in on this process, the work should already be done.
There was a problem hiding this comment.
Valid point, for now I have made this
# One of the easiest ways to do this, ...
I will however leave this conversation open and ask for further feedback from other docs contributors.
| overwrites[member] = member_overwrite | ||
|
|
||
| # and edit the channel with them: | ||
| await channel.edit(overwrites=overwrites) |
There was a problem hiding this comment.
I discussed this on other guide reviews, but I think it is always best practice to end a guide with prose as opposed to a codeblock/warning/other directive, because it makes the document feel better rounded off.
There was a problem hiding this comment.
I do agree, but I am not sure what to add here personally. Similar to the other remaining conversation I will await feedback from other contributors.
AbstractUmbra
force-pushed
the
docs/permissions-guide
branch
from
bcd33c9 to
7b5df4c
Compare
|
Is it possible to add documentation for how to do this? |
|
It's not possible to do that without OAuth2, which is outside of the scope of the library. Only administrators of the server are allowed to modify those permissions. What you can do however, is set the default permissions of a command. |
This comment was marked as off-topic.
This comment was marked as off-topic.
|
The documentation for it resides here. Though this is off-topic to this PR. Consider joining the discord.py server instead of asking questions here. |
AbstractUmbra
force-pushed
the
docs/permissions-guide
branch
from
7b5df4c to
4301b49
Compare
Summary
This PR adds a section to the guide for Permissions and PermissionOverwrites
Checklist