feat: added discord auth provider #7
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Thank you for the PR @justserdar You don't need to use ngrok actually, you can simply have HTTPS with Nuxt in development with We should not have a variable for the |
|
Hey! I removed the domain usage and went for the existing request path route as per the other oauth providers. |
samulefevre
pushed a commit
to samulefevre/nuxt-auth-utils
that referenced
this pull request
Nov 11, 2023
* feat: discord auth provider * Update discord.get.ts Typo Github -> Discord * fix: Update redirectUrl to include server route * refactor: cleanup discord provider * fix: Added discord env vars in example env * feat: Module addition of Discord vars * [autofix.ci] apply automated fixes * refactor: removed domain env usage * chore: update --------- Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com> Co-authored-by: Sébastien Chopin <[email protected]>
atinux
added a commit
that referenced
this pull request
Nov 13, 2023
* add oauth battle.net * add battle.net to env.example * fix: rm emailRequired and if cn to better place * feat: added discord auth provider (#7) * feat: discord auth provider * Update discord.get.ts Typo Github -> Discord * fix: Update redirectUrl to include server route * refactor: cleanup discord provider * fix: Added discord env vars in example env * feat: Module addition of Discord vars * [autofix.ci] apply automated fixes * refactor: removed domain env usage * chore: update --------- Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com> Co-authored-by: Sébastien Chopin <[email protected]> * chore: update --------- Co-authored-by: h+ <[email protected]> Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com> Co-authored-by: Sébastien Chopin <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Created a Discord provider based on the spotify provider.
The Discord provider is useful when you want to connect your Nuxt 3 project with your Discord Community.
Have information about your users displayed in your codebase or just simply create cool widgets or things visitors or users can see.
The auth flow requires a HTTPS redirect uri, I have it working using NGROk locally.
Discord provides a developer section where you can setup your applications and get your token information.
The user will have to setup their oauth settings at Discord like any other platform.
I found a challenge when introducing the redirect_uri as a configurable variable in the config because it needs to redirect back to 'https://your-domain.com + /auth/discord'
Edit: Improved it with the domain variable