Skip to content

Conversation

@kamityx
Copy link
Member

@kamityx kamityx commented Dec 9, 2025

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.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

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 A
  • Test B

Test Configuration:

  • Server Version:
  • Client:
  • Operating System:

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I checked the PR checks reports
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

@github-actions github-actions bot added Area: Api api folder Area: Web web folder labels Dec 9, 2025
@kamityx kamityx changed the title Feat/discord feat/discord-integration Dec 9, 2025
@kamityx kamityx requested a review from Copilot December 9, 2025 02:24
@kamityx kamityx self-assigned this Dec 9, 2025
Copy link

Copilot AI left a 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.

@kamityx kamityx requested a review from Copilot December 10, 2025 00:05
Copy link

Copilot AI left a 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.

@sonarqubecloud
Copy link

@kamityx kamityx merged commit ab515d2 into main Dec 10, 2025
9 checks passed
@kamityx kamityx deleted the feat/discord branch December 10, 2025 00:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Api api folder Area: Web web folder

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants