-
Notifications
You must be signed in to change notification settings - Fork 3
feat/discord-integration #30
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
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.
Pull request overview
This PR implements Discord OAuth integration, allowing users to link their accounts with Discord for authentication and authorization purposes. The implementation includes a complete OAuth flow with Discord's API, database schema for storing OAuth connections, and a UI component for managing Discord account linking.
Key Changes:
- Added Discord OAuth flow with link, callback, and unlink endpoints
- Implemented Discord bot integration with slash commands and button interactions
- Created database schema for storing OAuth provider connections
Reviewed changes
Copilot reviewed 61 out of 62 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| apps/web/src/sections/account_details/discord/index.tsx | New React component for Discord OAuth management UI |
| apps/web/src/sections/account_details/index.tsx | Integrated Discord OAuth component into account details |
| apps/web/src/ui/Buttons/ButtonImageAsLink/index.tsx | Added fondamento-title class to button styling |
| apps/api/src/presentation/v1/routes/accounts/oauth/* | Created OAuth route handlers for Discord linking |
| apps/api/src/application/usecases/account/discord* | Implemented use cases for Discord OAuth operations |
| apps/api/src/application/services/accountOauth/index.ts | Core service handling Discord OAuth logic |
| apps/api/src/domain/repositories/accountOauth/index.ts | Repository for OAuth data persistence |
| apps/api/src/domain/clients/discord/* | Discord API client and bot implementation |
| apps/api/src/domain/discord/* | Discord bot commands, buttons, and orchestration |
| apps/api/src/infra/env/index.ts | Added Discord configuration environment variables |
| apps/api/src/infra/di/* | Registered Discord-related dependencies |
| apps/api/src/domain/modules/cookies/index.ts | Changed cookie sameSite from Strict to Lax for OAuth |
| apps/api/prisma/models/* | Added OAuth database schema and migrations |
| apps/api/package.json | Added discord.js, axios, and axios-retry dependencies |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
apps/api/src/presentation/v1/routes/accounts/oauth/discord/unlink/index.ts
Outdated
Show resolved
Hide resolved
…o verify link and status
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.
Pull request overview
Copilot reviewed 72 out of 73 changed files in this pull request and generated 8 comments.
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
apps/api/src/presentation/v1/routes/accounts/oauth/discord/unlink/index.ts
Outdated
Show resolved
Hide resolved
|



Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Behaviour
Actual
Do this and that doesn't happens
Expected
Do this and that happens
Fixes #issuenumber
Type of change
Please delete options that are not relevant.
How Has This Been Tested
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Test Configuration:
Checklist