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

feature: Add Get***Url methods #1906

Closed
wants to merge 1 commit into from
Closed

feature: Add Get***Url methods #1906

wants to merge 1 commit into from

Conversation

Shane4368
Copy link

SemVer: minor

  • Add Get***Url methods to allow image format and size to be specified
  • Add Lottie to enum ImageFormat
  • Deprecate URL properties

Reference: https://discord.com/developers/docs/reference#image-formatting-cdn-endpoints

* Add Lottie to enum ImageFormat

* Allow image format and size to be specified

* Deprecate URL properties

* Use string.Empty in class CDN
@quinchs
Copy link
Member

quinchs commented Nov 23, 2021

Implemented in #1923

@quinchs quinchs closed this Nov 23, 2021
@Shane4368
Copy link
Author

After looking at the code (example: L95), some of the features in this PR are still not implemented.

This PR was intended to deprecate all URL properties and replace them with methods so the format (example: png, gif, jpeg) and size (example: 32, 128, 1024) could be specified.
RestGuild#IconUrl would become RestGuild#GetIconUrl(ImageFormat, ushort) for example.

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

Successfully merging this pull request may close these issues.

2 participants