Skip to content
This repository has been archived by the owner on Dec 13, 2021. It is now read-only.

Added support for Discord Archive (and presence status) #86

Open
wants to merge 2 commits into
base: node
Choose a base branch
from
Open

Added support for Discord Archive (and presence status) #86

wants to merge 2 commits into from

Conversation

ZaptoInc
Copy link

Hello, this PR is to add the presence_status from Discord Archive API: api.discord.wf/v2/bots/:id using the flat endpoint: api.discord.wf/v2/bots/:id/flat

@Andre601
Copy link

The question I have here is: Is that really useful for BotBlock?
This particular list seems to be the only one that has these specific features, so why should it be added if only one of many lists is having this?

@cheesycod
Copy link
Contributor

cheesycod commented Mar 26, 2021

Fates List does have this feature too but it’s completely different in how it returns however. So I kinda agree with your point @Andre601 about not many lists having this in the correct format if at all having it

Also, maybe use unknown instead of null for default

@ZaptoInc
Copy link
Author

Fates List does have this feature too but it’s completely different in how it returns however. So I kinda agree with your point @Andre601 about not many lists having this in the correct format if at all having it

Also, maybe use unknown instead of null for default

yeah, i'll change that

@ZaptoInc
Copy link
Author

The question I have here is: Is that really useful for BotBlock?
This particular list seems to be the only one that has these specific features, so why should it be added if only one of many lists is having this?

Hello, v4 asked me to do a PR so i did

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants