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

Add more informations for application info #1341

Closed
wants to merge 6 commits into from

Conversation

laggron42
Copy link
Contributor

The discord API gives informations about RPC, the bot status ("public bot" ticked or not in the token settings) or if it requires code grant.
https://discordapp.com/developers/docs/topics/oauth2#get-current-application-information

These informations are not available on discord.py, so I added them as an attribute of client.AppInfo.

This adds the rpc_origin, bot_public and bot_require_code_grant attributes
@Rapptz
Copy link
Owner

Rapptz commented Jun 11, 2018

This needs documentation to be merged.

@laggron42
Copy link
Contributor Author

Alright, added the docs and fixed the conflict. I though you were using sphinx autodoc, so I was a bit lost...

docs/api.rst Outdated
.. attribute:: bot_require_code_grant

``True`` if the bot requires code grant, else ``False``.
Corresponf to the "Bot requires code grant" bow in the token
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you fix these typos too?

@Rapptz Rapptz added the merged Tag to mark pull requests as rebased or merged label Aug 23, 2018
@Rapptz Rapptz closed this Aug 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged Tag to mark pull requests as rebased or merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants