Giter Club home page Giter Club logo

Comments (30)

Rapptz avatar Rapptz commented on June 1, 2024 9

What about roles that are there only for colour?

from discord-api-docs.

ducc avatar ducc commented on June 1, 2024 5

As @kantenkugel said above, something like ALL_PERMISSIONS would be ideal for bots that need to moderate.

from discord-api-docs.

vishnevskiy avatar vishnevskiy commented on June 1, 2024 4

You will have to remove colors from your mod roles and just add them to the bottom ones if you want to continue doing that. Role hierarchy is one of the top requested features we have by far so we have to make this change.

from discord-api-docs.

vishnevskiy avatar vishnevskiy commented on June 1, 2024 4

@kantenkugel I guess it was not, but we should make it one. I kinda assumed it was the highest role with a color. Will consult the team.

from discord-api-docs.

khionu avatar khionu commented on June 1, 2024 3

\o/ Long awaited, this is fantastic!! Thank you soooooo much!!!!

from discord-api-docs.

srhinos avatar srhinos commented on June 1, 2024 3

This will severely harm RH1-N0's ability to manage servers. Its already pulling teeth to have perms right on servers where the role is already created for them. Now I'll have to explain to servers that RH1 needs to sit at either the highest role or higher than all other moderator roles.

Most servers sort RH1 as the lowest as they display him on the sidebar so people know he's there and being used but don't want a bot sitting at the top.

On top of that, many servers use colors as a role and my two other bots that do this on wow and hots. Though this doesn't wreck it, it'll make people with roles (moderators, staff, mvps) not able to set their class or region colors which sucks a little.

from discord-api-docs.

kantenkugel avatar kantenkugel commented on June 1, 2024 3

This also requires the owner to give new permissions like the nick permission to the admin-roles in order for them to be able to use them (as there is no more permission to grant all others).
Would it be possible to keep the Permission with the MANAGE_ROLES offset and rename it to something like: ALL_PERMISSIONS (keep functionality the same, and just add a new MANAGE_ROLES)?

from discord-api-docs.

kantenkugel avatar kantenkugel commented on June 1, 2024 2

@RogueException because theres multiple scenarios (like co-owner) where its easier to give all perms with one flag than to manually add all permissions (including the work needed to add future permissions). It also makes some Manage-stuff easier (for bots that you actually want to grant all perms (regarding of his role-position)

from discord-api-docs.

khazhyk avatar khazhyk commented on June 1, 2024 2

I have roles that are used only for color that are necessarily on the very top of the list. From my understanding, if I grant someone manage_roles and give them a color role (with no permissions) they can add to any role/any member?

Position may also be odd since it has the dual use of sorting the sidebar - I can echo rhino's concern that many servers I've seen put bots lower than staff or other roles so they aren't first on the sidebar, but still want to show them separately.

This was suggested in chat:
What if, you could manage roles, for roles in a lower position than the one granting manage_roles? (Same for kick/ban/etc.), that way assigning a role with zero permissions wouldn't affect anything.

from discord-api-docs.

Azareal avatar Azareal commented on June 1, 2024 2

ALL_PERMISSIONS would be a great feature for a co-owner / admin type thing.

One way to improve it further is if it overrode the per-channel permissions (kinda like how owner does for a sort of superuser role), as it can be a big hassle fixing channels one by one when you discover that your admins don't have manage roles in a channel thanks to an obscure overlap of permissions.

from discord-api-docs.

vishnevskiy avatar vishnevskiy commented on June 1, 2024 2

I have decided to keep the old bit as a SUPER_USER permission that is basically an owner but cannot delete the server and is subject to role hierarchy for managing roles, ban, kicks.

from discord-api-docs.

vishnevskiy avatar vishnevskiy commented on June 1, 2024 1

@cookkkie That is part of the point. Bots should start just above @everyone and be moved by the owner. You should either render on your site or respond in messages if the bot's role is too low. Maybe we will add this to OAuth2 page later so they can choose in advance, but there is no way for your bot to really know the position it wants since every server is different.

from discord-api-docs.

vishnevskiy avatar vishnevskiy commented on June 1, 2024 1

@ayust yes

from discord-api-docs.

vishnevskiy avatar vishnevskiy commented on June 1, 2024 1

@Bubka3 its built, hoping to release this week.

from discord-api-docs.

cookkkie avatar cookkkie commented on June 1, 2024

So, how to set a position for a role? Like for bots when in the authorize page?

from discord-api-docs.

kantenkugel avatar kantenkugel commented on June 1, 2024

Will the MANAGE_PERMISSIONS on the channel-level also be ported to new offset?

from discord-api-docs.

DV8FromTheWorld avatar DV8FromTheWorld commented on June 1, 2024

I agree. ALL_PERMISSIONS would be better.

from discord-api-docs.

Auralytical avatar Auralytical commented on June 1, 2024

I don't understand why we need ALL_PERMISSIONS at all? It's a bitfield... you can just set every bit. Why is there a need for any alternative?

from discord-api-docs.

Auralytical avatar Auralytical commented on June 1, 2024

I agree with @Azareal, I'd be up for an ALL_PERMISSIONS if channel permissions couldn't revoke it. See: Discord API where channel owners (with manage permissions) are able to block admins from viewing/interacting their channel.

from discord-api-docs.

Fabricio20 avatar Fabricio20 commented on June 1, 2024

What @khazhyk said is basically my scenarion, I have roles with colors up on the list with no permissions and roles down the list with permissions.

from discord-api-docs.

ayust avatar ayust commented on June 1, 2024

Does this change also mean that the new Manage Roles will no longer imply having read access to all channels on the server?

from discord-api-docs.

kantenkugel avatar kantenkugel commented on June 1, 2024

oh nice... so removing color is a thing? (letting below roles determine color)

from discord-api-docs.

Bubka3 avatar Bubka3 commented on June 1, 2024

What type of timeframe do we have to implement this change (approx release frame, if possible)?

from discord-api-docs.

kwando1313 avatar kwando1313 commented on June 1, 2024

Will managing nicknames on the server follow the logic of the new manage roles permission?

from discord-api-docs.

vishnevskiy avatar vishnevskiy commented on June 1, 2024

@kwando1313 will work like ban/kick, lower rank only.

from discord-api-docs.

TobiTenno avatar TobiTenno commented on June 1, 2024

Will having the owner being a "supreme" level of permissions make anyone with the "Manage Roles" perm unable to change the guild owner's nickname, all other levels being the same?

from discord-api-docs.

vishnevskiy avatar vishnevskiy commented on June 1, 2024

yes the owner cannot be touched currently

from discord-api-docs.

khionu avatar khionu commented on June 1, 2024

Actually, that's only true through the App. The API permits it, a friend did it through their SelfBot

Amendment: That's on their word, but Zeta hasn't given me a reason to disbelieve them before shrugs

from discord-api-docs.

khionu avatar khionu commented on June 1, 2024

Further testing: API works, RightClicking works, denied through the Memberlist

from discord-api-docs.

kantenkugel avatar kantenkugel commented on June 1, 2024

Would you consider running a server-side patch that moves the old MANAGE_PERMISSIONS (channel-side) to the new offset? Cuz at the moment, the Channel-override just shows no overrides, but the API shows the old flag (now only used by ADMIN) which doesn't make sense on channel-side.

from discord-api-docs.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.