Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BE] AOP 적용 #181

Merged
merged 9 commits into from
Nov 29, 2024
Merged

[BE] AOP 적용 #181

merged 9 commits into from
Nov 29, 2024

Conversation

urinaner
Copy link
Owner

  • 💯 테스트는 잘 통과했나요?
  • 🏗️ 빌드는 성공했나요?
  • 🧹 불필요한 코드는 제거했나요?
  • 💭 이슈는 등록했나요?
  • 🏷️ 라벨은 등록했나요?

작업 내용

AOP적용, 어노테이션 생성, 페이지 리펙토링
페이지 리펙토링
config 정리

스크린샷

아래 사진은 사용예시에요
스크린샷 2024-11-28 오후 7 31 01
스크린샷 2024-11-28 오후 7 31 08
스크린샷 2024-11-28 오후 7 31 21
스크린샷 2024-11-28 오후 7 34 25

사용법입니다. 로그인이 필요한 controller에 @authuser 삽입 -> User 정보 도출 가능 -> 예약 화면에 학번, 이름 가져올 수 있는 기능.
예외처리는 AuthException 참고.
페이징처리 이해하기 쉽고 다양한 정렬조건 가능하게 변경.

주의사항

PageExceptionHandler로 예외처리가 잘 안되는듯함.
UserController에서 로그인 2번째부터 안됨. �user 등록하기 전 userRepository에 같은 학번이 존재하는지 체크 후 save 로직 작성 필요.
추후 관리자 AOP 작성 예정.
Closes #180

@urinaner urinaner added enhancement New feature or request 😄 BE refactor labels Nov 28, 2024
@urinaner urinaner requested a review from 2Jin1031 November 28, 2024 11:07
@urinaner urinaner self-assigned this Nov 28, 2024
@2Jin1031
Copy link
Collaborator

와 AOP! 고생하셨습니다 👍👍

@2Jin1031 2Jin1031 merged commit 61a76d5 into main Nov 29, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BE] AOP 적용
2 participants