Skip to content

Release 1.7.0 - #92

Merged
welllucky merged 0 commit into
mainfrom
dev
Aug 24, 2025
Merged

welllucky merged 0 commit into
mainfrom
dev

Conversation

@welllucky

Copy link
Copy Markdown
Owner

No description provided.

@welllucky welllucky self-assigned this Aug 24, 2025
Copilot AI review requested due to automatic review settings August 24, 2025 08:59
@welllucky welllucky added the enhancement New feature or request label Aug 24, 2025
@welllucky
welllucky enabled auto-merge (rebase) August 24, 2025 09:00
@deepsource-io

deepsource-io Bot commented Aug 24, 2025 •

Copy link
Copy Markdown

Here's the code health analysis summary for commits 909a1e1..f544c41. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript❌ Failure
❗ 12 occurences introduced
🎯 5 occurences resolved
View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This is a major refactoring release (v1.7.0) that restructures the application architecture, introduces new modules, and enhances API documentation. The key changes include reorganizing the module structure with new shared/access/core separation, implementing comprehensive Swagger documentation, and introducing account-based authentication.

  • Module architecture restructuring with clear separation between core, access, and backoffice modules
  • Introduction of Account entity and associated services to separate user authentication from user data
  • Comprehensive Swagger/OpenAPI documentation across all endpoints
  • Migration from repository pattern to modular service architecture

Reviewed Changes

Copilot reviewed 114 out of 121 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
tests/role/role.e2e.ts Updates role change tests to work with new Account service architecture
src/utils/validators/ Updates import paths to use new shared module structure
src/utils/functions/ Removes deprecated utility functions and updates configuration
src/typing/dto/ Adds comprehensive API documentation with Swagger decorators
src/services-application.ts New application bootstrap class with separated Swagger documentation
src/repositories/index.ts Removes centralized repository exports in favor of modular approach
src/modules/shared/ New shared module structure with validation, user, session, and other services
src/modules/core/ Restructured core modules with updated imports and API documentation
src/modules/backoffice/ Enhanced backoffice modules with role-based access and Swagger docs
src/modules/access/ New access module containing authentication and account management
src/models/ Updates model classes to work with new Account entity structure
src/main.ts Simplified main file using new ServicesApplication bootstrap class

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment on lines 59 to +61
sessionId,
register,
sessionData,

This comment was marked as resolved.

@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
15.6% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@welllucky
welllucky merged commit f544c41 into main Aug 24, 2025
1 of 3 checks passed
@welllucky
welllucky deleted the dev branch August 24, 2025 09:09
@github-project-automation github-project-automation Bot moved this to Done in Services Aug 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants