platformplatform / PlatformPlatform

Alpha state. A platform designed for building enterprise-grade, multi-tenant products using Azure, .NET, React, TypeScript, Infrastructure as Code, etc.
https://platformplatform.github.io
MIT License
253 stars 26 forks source link

Refactor dependency configuration and naming for improved clarity #623

Closed tjementum closed 5 days ago

tjementum commented 5 days ago

Summary & Motivation

Introduce a series of small refactorings to improve code clarity and maintainability. Dependency configuration has been updated to use fluent interface chaining, and related classes in the SharedKernel have been moved to a Configuration namespace. The AddApiServices method is renamed to AddWorkerServices for better clarity.

Other changes include renaming AddEmailSignatureService to AddEmailSignatureClient, aligning IdGenerators in SharedKernel.Tests with StronglyTypedIds in the SharedKernel, and refactoring dependency configuration in the AppGateway.

The GetLoggedInUserAsync method now throws an exception if no user is logged in, enforcing stricter behavior. Additionally, custom properties on the ChangeLocale event are renamed from old_locale/new_locale to from_locale/to_locale, aligning with the conventions used in the ChangeUserRole event.

Checklist

linear[bot] commented 5 days ago

PP-248 Refactor dependency configuration and naming for improved clarity

sonarcloud[bot] commented 5 days ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
79.6% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud