Skip to content

Conversation

@hackermondev
Copy link
Contributor

The URL that is shows in the docs is the old url for applications. In this pull request I updated the URL to the right url.

Please describe the changes this PR makes and why it should be merged:
The URL that is shows in the docs is the old url for applications. In this pull request I updated the URL to the right url. In the future Discord might show a 404 page for that url so everything needs to be ready and updated.

Status

  • [ Yes ] Code changes have been tested against the Discord API, or there are no code changes
  • [ Yes ] I know how to update typings and have done so, or typings don't need updating

Semantic versioning classification:

  • [ No ] This PR changes the library's interface (methods or parameters added)
    • [ No ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)
  • [ Yes ] This PR only includes non-code changes, like changes to documentation, README, etc.

The URL that is shows in the docs is the old url for applications. In this pull request I updated the URL to the right url.
@vaporoxx
Copy link
Contributor

vaporoxx commented Aug 13, 2020

This link is present multiple times in the library and should be changed in each file, I have listed them here [deleted]

@hackermondev
Copy link
Contributor Author

@Vaporox done

@iCrawl iCrawl changed the title Updated URL docs: updated applications URL Aug 14, 2020
@iCrawl iCrawl merged commit 0b38c5d into discordjs:master Aug 14, 2020
@hackermondev hackermondev deleted the patch-1 branch August 17, 2020 19:22
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.

4 participants