Skip to content
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

Bump Discord API version to version 9 and add more message type to be deleted #223

Merged
merged 3 commits into from
Apr 26, 2022

Conversation

VictorienXP
Copy link
Contributor

@VictorienXP VictorienXP commented Jul 5, 2021

Discord clients are now using the version 9 of the Discord API.

Version 9 is required in order to be able to use Threads.

This pull request also adds more message types to be deleted (≥6 & ≤21).
image
As far as I know they are all messages types that can be deleted without any issue.


Please note that the script currently does not work on Threads as they don't seem to be indexed.

{"message": "Missing Access", "code": 50001}

I'm not sure if it's a bug or a limitation of the early access.
Anyway this is something to look for as Threads aren't a released feature yet.


This change is Reviewable

@VictorienXP
Copy link
Contributor Author

I forgot to mention that ever since the threads feature officially launched, deleting messages from them works fine (if they aren't archived).
Messages from threads are indexed in the channel search itself.

@victornpb victornpb added the PR looks good Looks like it's good for merging (maybe be in the next release) label Apr 26, 2022
@victornpb victornpb merged commit a5c3e13 into victornpb:master Apr 26, 2022
@VictorienXP VictorienXP deleted the bump-discord-api-version branch April 28, 2022 01:18
victornpb added a commit that referenced this pull request Apr 29, 2022
Bump Discord API version to version 9 and add more message type to be deleted  PR looks good (#223 by VictorienXP)
Add wildcard subdomain for discord.com match on the userscript (#224 by VictorienXP)
Fix bug where process is prematurely stopped due to receiving a full array of skippedMessages (#323 by aijorgenson)
fix: stricter url matching (#335 by SethFalco)
refactor: reduce number of @matchs (#336 by SethFalco)
Removed duplicate OR operator (#338 by aydinyal)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR looks good Looks like it's good for merging (maybe be in the next release)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants