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
feat: Added User.avatar_url() to get the resulting URL from a user #515
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
will this conflict with bot.user at all? |
This does not conflict with bot.user/client.user (It just adds a property) Anyway, this looks good to me, I'm merging :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another LGTM!
) * feat: Added User.avatar_url() to get the resulting URL from a user * ci: correct from checks. * fix: Make the User.avatar_url a property * fix: remove useless import * fix: remove wrong headers Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: DeltaX <33706469+DeltaXWizard@users.noreply.github.com>
About
This pull request is about creating a
User.avatar_url()
function, to return the proper User avatar URL.Checklist
pre-commit
to format and lint the change(s) made.3.8.6
and higher.