Conversation
|
Thanks for opening this pull request! This pull request can be checked-out with: git fetch origin pull/3252/head:pr-3252
git checkout pr-3252This pull request can be installed with: pip install git+https://github.com/Pycord-Development/pycord@refs/pull/3252/head |
Lulalaby
left a comment
There was a problem hiding this comment.
Can someone request changes reminding me to add editing & creating channels with that flag
and whatever else i may have forgotten
Soheab
left a comment
There was a problem hiding this comment.
Needs a way for bots to create this kind of channel
This message was sponsored by toothyfernsan on discord.
|
I tested it with all but MediaChannels since I don't have access to those anywhere, but I'd still like a second pair of testing eyes on this |
|
Also, creating a channel with this turned on isn't possible unfortunately, gotta create + edit |
# Conflicts: # CHANGELOG.md
Signed-off-by: Lala Sabathil <lala@pycord.dev>
Signed-off-by: Lala Sabathil <lala@pycord.dev>
This comment was marked as resolved.
This comment was marked as resolved.
JustaSqu1d
left a comment
There was a problem hiding this comment.
Inconsistency between notes and warnings for the documented behavior regarding NSFW and spoiler flag.
vmphase
left a comment
There was a problem hiding this comment.
LGTM from the technical side.
Co-authored-by: Lala Sabathil <lala@pycord.dev> Signed-off-by: ToothyDev <55001472+ToothyDev@users.noreply.github.com>
|
Thanks ! |
vmphase
left a comment
There was a problem hiding this comment.
Why does this PR add any kind of support for spoiler on StageChannel, while documentation explicitly says:
IS_SPOILER_CHANNEL... Can be set on all textual guild channels and voice channels (not GUILD_STAGE).
To stay internally consistent, because NSFW also doesn't work on stage channels, but Pycord supports it, it CAN be passed, it just has no effect. And actually, both toggles do exist (and work) in the client UI. Basically, everything works except it having an actual effect on the channel. So, while yes, it doesn't actually do anything, I still think it's worth keeping for internal consistency and because Discord might eventually make it work for stage channels (same for nsfw) |
| reason: str | None = None, | ||
| category: CategoryChannel | None = None, |
There was a problem hiding this comment.
Even though there is no technical difference, I would rather tend to making these ... for consistency with other overloads PEP 484 stub conventions..
| category: CategoryChannel | None = None, | ||
| reason: str | None = None, |
There was a problem hiding this comment.
Same as in create_voice_channel overload.
I see. If the development team is fine with this, sure. |
Summary
This adds
ChannelFlags.is_spoiler_channel(following https://docs.discord.food/resources/channel#channel-flags) as well as a.spoilerproperty to all applicable channel types (all except category). Same goes for making it possible to create these channels via the regular channel creation methods, again with the exception of category channels. Channel editing with this flag is also functional.Now that it's officially documented, here's the ddocs PR: discord/discord-api-docs#8465
Depends on discord/discord-api-docs#8465
Information
examples, ...).
Checklist
type: ignorecomments were used, a comment is also left explaining why.