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

pullrequest: 프로젝트 1차 리팩토링 #96

Closed
wants to merge 2 commits into from

Conversation

juwon-code
Copy link
Collaborator

#️⃣연관된 이슈

See also: #95
jobPost와 Apply, ScoutRequest를 제외한 도메인의 컴포넌트들을 리팩토링합니다.

☑️ PR 유형

  • 새로운 기능 추가
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더 삭제
  • 파일 혹은 폴더명 수정

📝 작업 내용

로컬 DB 연결 설정 제거

로컬 데이터베이스 연결 설정이 공유 설정 파일에 들어있어 제거합니다.

Commit: 8953b82

미디어 컴포넌트 리팩토링

  • 회원 및 일거리 도메인과 1:1로 연결합니다.
  • 타 도메인에 필요한 업로드 및 삭제 메소드를 추가합니다.
  • 이미지 및 비디오 검사 규칙을 세부 설정합니다.

Commit: f44bd5c

시큐리티 및 토큰 코드 리팩토링

  • security -> token으로 패키지 변경 및 클래스 명을 변경합니다.
  • 토큰과 회원 서비스 메소드 분리를 명확하게 분리합니다.
  • 토큰 요청 및 응답 DTO Swagger 주석 추가합니다.
  • deprecated된 JWT 생성 설정 최신화 헙나다,
  • 시큐리티 필터 및 제공 클래스 리팩토링 합니다.

Commit: 5016341

회원 컴포넌트 리팩토링

  • 회원 수정 및 삭제 메소드 실행 전에 권한을 확인하도록 설정합니다.
  • 용도별 DTO 분리 및 변환 역할을 수행하는 매퍼 클래스 추가합니다.
  • 메소드명 및 URI RESTful하게 변경합니다.

Commit: 4b30314

피드 컨트롤러 메소드 권한 설정

피드 수정 및 삭제 메소드 실행 전에 권한을 확인하도록 설정합니다.

Commit: 168dca5

멤버 제공 컴포넌트 위치 변경

변경된 MemberProvider 위치가 변경됨에 따라 Import 참조 조수를 변경합니다.

Commit: 3630233

✅ 피드백 반영사항

회의에서 말씀드린대로 피드와 회원의 특정 메소드의 실행 전에 권한을 확인하고, 파일 이름을을 변경하려고 합니다.

💬 리뷰 요구사항

밤을 새워서 리팩토링을 수행했기에, 새벽부터 집중력이 떨어져서 누락된 부분이 있을 수 있습니다. 또한,
JobPost와 Apply, ScoutRequest를 제외한 대부분의 코드가 구조부터 변경되었으므로, 최신화 Merge가 필요합니다.

Copy link
Collaborator

@ghtndl ghtndl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다!~

@juwon-code juwon-code closed this Oct 8, 2024
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