Skip to content
This repository has been archived by the owner on Apr 20, 2022. It is now read-only.

Fixed applications not containing the TOS and Privacy strings. #382

Merged
merged 3 commits into from
Dec 19, 2021
Merged

Fixed applications not containing the TOS and Privacy strings. #382

merged 3 commits into from
Dec 19, 2021

Conversation

FeroxFoxxo
Copy link
Contributor

@FeroxFoxxo FeroxFoxxo commented Dec 18, 2021

Fixed Application Modal

This is a PR fixing the application modal. It adds two fields, the TOS and Privacy Policy, fields that are in Discord's API but not DNet's.

Related issue: discord-net/Discord.Net#1968

src/Discord.Net.Core/Entities/IApplication.cs Outdated Show resolved Hide resolved
Copy link
Member

@quinchs quinchs left a comment

Choose a reason for hiding this comment

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

mean't to request changes

@MrCakeSlayer MrCakeSlayer added Enhancement New feature or request Internal Somthing internal that needs to be changed labels Dec 18, 2021
Copy link
Contributor Author

@FeroxFoxxo FeroxFoxxo left a comment

Choose a reason for hiding this comment

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

Fixed all the issues given and added documentation for flags rather than removing the new line.

@quinchs quinchs merged commit 7e8f198 into Discord-Net-Labs:release/3.x Dec 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Enhancement New feature or request Internal Somthing internal that needs to be changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants