Skip to content

Conversation

@aalmada
Copy link
Owner

@aalmada aalmada commented Jan 13, 2026

Pull Request Template

Description

Refactor soft-deletion code. Add indexes.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Test addition/update

Related Issues

Fixes #

Changes Made

Testing

  • Unit tests pass (dotnet test)
  • Build succeeds (dotnet build)
  • Code formatting is correct (dotnet format --verify-no-changes)
  • No new analyzer warnings
  • Manual testing completed

Checklist

  • My code follows the project's coding standards
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

Screenshots (if applicable)

Additional Notes

@aalmada aalmada force-pushed the fix/soft_deletion_2 branch from 40aee95 to d0bbacc Compare January 13, 2026 09:11
@github-actions
Copy link

Code Coverage

Package Line Rate Branch Rate Health
BookStore.Shared 14% 5%
BookStore.ApiService 11% 15%
Summary 11% (830 / 7466) 14% (328 / 2292)

@aalmada aalmada merged commit 6974f0b into main Jan 13, 2026
4 checks passed
@aalmada aalmada deleted the fix/soft_deletion_2 branch January 13, 2026 14:18
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.

2 participants