Skip to content

Refactor and enhance services, security, and AI integration#75

Merged
samuelzedec merged 1 commit intomainfrom
develop
Sep 12, 2025
Merged

Refactor and enhance services, security, and AI integration#75
samuelzedec merged 1 commit intomainfrom
develop

Conversation

@samuelzedec
Copy link
Owner

Changes Introduced

This pull request includes the following updates:

  1. Refactoring and Naming Improvements:

    • Simplified test class names by aligning with updated service/feature naming conventions.
    • Removed unnecessary dependencies from LocalImageStorageService.
  2. Password Validation:

    • Enhanced validation rules for passwords during company and user creation.
  3. Amazon S3 Service Integration:

    • Added AmazonS3Service for robust image storage in AWS S3.
    • Improved error handling using standardized error messages.
  4. Namespace and Dependency Injection Refactoring:

    • Updated namespaces for clarity, improving service and domain alignment.
    • Conditional registration of local vs AWS services based on the environment.
  5. Semantic Kernel Features:

    • Integrated Semantic Kernel packages for leveraging AI capabilities.
    • Enabled PostgreSQL pgvector extension to facilitate semantic search.
  6. Documentation and Configuration Updates:

    • Enhanced .gitignore for better storage folder management.
    • Updated appsettings.json

…ature naming conventions

- Removed redundant "UnitTests" suffix from test class names.
- Updated namespaces and test class names to reflect recent service and feature renaming.
- Adjusted `LocalImageStorageService` constructor to remove `IWebHostEnvironment` dependency and calculate storage path based on current directory.
- Updated `.gitignore` to exclude `storage` directory content except `.gitkeep` files.
@samuelzedec samuelzedec merged commit 2dded18 into main Sep 12, 2025
1 check passed
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.

1 participant