Skip to content

Facade migration t3 lector 83 - #129

Closed
DjesikaV wants to merge 6 commits into
mainfrom
Facade-Migration-T3-lector-83
Closed

Facade migration t3 lector 83#129
DjesikaV wants to merge 6 commits into
mainfrom
Facade-Migration-T3-lector-83

Conversation

@DjesikaV

Copy link
Copy Markdown
Contributor

Changes:

  • Added LectorWebFacade
  • Added LectorValidator
  • Updated LectorController to inject only the facade
  • Refactored LectorService to use only the ServiceRepository
  • Removed @transactional from LectorService
  • Updated LectorMapper to map request and response Dtos directly
  • Removed validation annotations from LectorResponseDto
  • Updated LectorServiceTest
  • Added LectorWebFacadeTest
  • Added LectorValidatorTest

@DjesikaV
DjesikaV requested a review from a team as a code owner July 16, 2026 08:43
@DjesikaV DjesikaV closed this Jul 16, 2026
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