[ReadState] Deprecation for Bots - Githubissues

discord / discord-api-docs

Official Discord API Documentation

https://discord.com/developers/docs/intro

Other

5.9k stars 1.25k forks source link

[ReadState] Deprecation for Bots #13

Closed jhgg closed 8 years ago

jhgg commented 8 years ago

We're disabling read states entirely for bots. This shouldn't break any APIs (yet - read on for more).

Planned:

[X] The read state array in the READY packet will return an empty array for bots
[X] The message_ack endpoint will return 401 if you are trying to ack a message on a bot account.
[X] Sending messages will no longer implicitly self ack the channel the message was sent in.
Breaking Changes:
We're going to remove read_state entirely from the ready packet in a few weeks. Update your libraries to not depend on this!
OK Libraries
[x] Discord.net
[x] Discord Sharp
[x] Go - DiscordAPI
[x] Go - DiscordGo
[x] Discord4j
[x] Javacord
[x] JDA
[x] Discord.io
[x] Discord.js
[x] Discordie
[x] DiscordPHP
[x] Discord.py
[x] Discordrb
[x] Discord.rs

TODO: Document this.

vishnevskiy commented 8 years ago

I think they should just not receive the array at all in the next version bump

iopred commented 8 years ago

We hadn't planned on removing this from our API as we have a number of clients built on discordgo. Seems reasonable to add some documentation though.

jhgg commented 8 years ago

@iopred that's fine - you don't have to remove it from your lib, just need to make sure it doesn't blow up if it's not present for bot users.

bwmarrin commented 8 years ago

It won't blow up @jhgg.

suicvne commented 8 years ago

DiscordSharp never touched read_state

jhgg commented 8 years ago

I've confirmed that all libs should function with this removed. Will make that change on May 1st.