Skip to content

Fix migration rollback and improve validation and documentation#71

Merged
samuelzedec merged 4 commits intomainfrom
develop
Sep 6, 2025
Merged

Fix migration rollback and improve validation and documentation#71
samuelzedec merged 4 commits intomainfrom
develop

Conversation

@samuelzedec
Copy link
Owner

### Description

- Restored logic to drop Quartz tables during migration rollback for consistency.
- Added `IsInEnum` validation to ensure `Position` is within the enum range in `CreateUserCommandValidator`.
- Updated Dependency Injection to allow internal types for validator discovery.
- Added XML documentation for `PermissionPolicyProvider` and `RequirePermissionAttribute` to clarify dynamic authorization and usage.

### Checklist

- [ ] Tests added for changes
- [ ] Documentation updated (if needed)
- [ ] Code follows coding standards
- [ ] Ready for review

…ePermissionAttribute`

- Added summary and remarks to clarify the dynamic authorization generation in `PermissionPolicyProvider`.
- Documented the usage and flow of `RequirePermissionAttribute` for permission-based authorization.
- Updated `CreateUserCommandValidator` to include `IsInEnum` validation for `Position` property.
- Restored missing SQL logic to drop Quartz tables during migration rollback for consistency and cleanup.
@samuelzedec samuelzedec merged commit 7a0dc03 into main Sep 6, 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