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

Typings: Renamed dynamicIconURL in User to dynamicAvatarURL #373

Merged
merged 2 commits into from
Mar 20, 2018

Conversation

TheAkio
Copy link
Contributor

@TheAkio TheAkio commented Mar 20, 2018

By accident I noticed that the function dynamicIconURL() for User is defined in the typings but it's not really in the source code.

@Pepijn98
Copy link

Actually that doesn't have to be removed, just renamed to dynamicAvatarURL. I guess it was copy pasted from somewhere else or a typo.

@TheAkio
Copy link
Contributor Author

TheAkio commented Mar 20, 2018

Yep lol... Should have checked if dynamicAvatarURL existed.

@TheAkio TheAkio changed the title Removed typings for dynamicIconURL in User because it doesn't exist Typings: Renamed dynamicIconURL in User to dynamicAvatarURL Mar 20, 2018
@abalabahaha abalabahaha merged commit 09e420c into abalabahaha:dev Mar 20, 2018
nandub added a commit to nandub/eris that referenced this pull request May 22, 2018
* upstream/master:
  0.8.6
  Bump ws dependency
  Prevent command overwriting in CommandClient (abalabahaha#389)
  Fix PermissionOverwrite docs, lint
  Add role.mentionable to guildRoleUpdate (abalabahaha#387)
  Remove global reaction ratelimit (abalabahaha#388)
  Update Message.cleanContent (fix abalabahaha#377) (abalabahaha#378)
  Send WS close code 1000 on Shard.disconnect() (abalabahaha#383)
  Change how duplicate presences are detected (abalabahaha#376)
  Fix OggOpusTransformer for multi-track files (abalabahaha#380)
  Added closeVoiceConnection() (abalabahaha#368)
  Add new system messages (abalabahaha#375)
  Remove ImageSizes array and fix minimum image size
  Typings: Renamed dynamicIconURL in User to dynamicAvatarURL (abalabahaha#373)
nandub added a commit to nandub/eris that referenced this pull request May 22, 2018
* upstream/master:
  0.8.6
  Bump ws dependency
  Prevent command overwriting in CommandClient (abalabahaha#389)
  Fix PermissionOverwrite docs, lint
  Add role.mentionable to guildRoleUpdate (abalabahaha#387)
  Remove global reaction ratelimit (abalabahaha#388)
  Update Message.cleanContent (fix abalabahaha#377) (abalabahaha#378)
  Send WS close code 1000 on Shard.disconnect() (abalabahaha#383)
  Change how duplicate presences are detected (abalabahaha#376)
  Fix OggOpusTransformer for multi-track files (abalabahaha#380)
  Added closeVoiceConnection() (abalabahaha#368)
  Add new system messages (abalabahaha#375)
  Remove ImageSizes array and fix minimum image size
  Typings: Renamed dynamicIconURL in User to dynamicAvatarURL (abalabahaha#373)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants