Skip to content

Conversation

@hnjm
Copy link
Owner

@hnjm hnjm commented Nov 3, 2024

update to 2024-11-03

AliRafay and others added 4 commits October 19, 2024 08:58
* Refactor and consolidate Shared project structure

The project structure has been modified to remove the "Shared" project from the "api\modules" directory and add a new "Shared" project in the "Shared" directory. The solution configuration has been updated accordingly. References to the old "Shared" project have been replaced with references to the new "Shared" project.

The `TenantConstants` and `IdentityConstants` classes have been moved to the `FSH.Starter.Shared.Authorization` namespace. The `IdentityDbInitializer` and `TokenService` classes have been updated to use the new constants. The `ClaimsPrincipalExtensions` class has been removed.

Various endpoint and service classes have been updated to use the new `FSH.Starter.Shared.Authorization` namespace. The `Core.csproj` and `Infrastructure.csproj` files have been updated to reference the new "Shared" project.

The `FSH.Framework.Core.Tenant` and `FSH.Framework.Infrastructure.Tenant` namespaces and their associated classes have been removed, and the `FSH.Starter.Shared.Authorization` namespace has been added. Unused namespaces have been removed from multiple files.

The `Shared.csproj` file was updated to target .NET 8.0 and enable implicit usings and nullable reference types. New static classes for constants related to image formats, claims, actions, permissions, resources, roles, identity, and tenants have been added.

* Refactor: Rename FshAction and FshResource classes

Renamed `FshAction` to `FshActions` and `FshResource` to `FshResources` across multiple files for consistency:
- Updated class names in `FshActions.cs` and `FshResources.cs`.
- Modified references in `FshPermissions.cs`, `EntityTableContext.cs`, and various `.razor.cs` files.
- Ensured all permission checks and resource assignments use the new class names.
@hnjm hnjm merged commit 7cb3d1c into hnjm:main Nov 3, 2024
4 checks 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.

4 participants