Skip to content

GUIDE-63 error코드 관련 기초 설정 및 파일 작성#7

Merged
2ralumin merged 25 commits intoguiderun-project:developfrom
JaeGeon-SMU:GUIDE-62-어드민-기능-백엔드-작업
Jan 19, 2024

Hidden character warning

The head ref may contain hidden characters: "GUIDE-62-\uc5b4\ub4dc\ubbfc-\uae30\ub2a5-\ubc31\uc5d4\ub4dc-\uc791\uc5c5"
Merged

GUIDE-63 error코드 관련 기초 설정 및 파일 작성#7
2ralumin merged 25 commits intoguiderun-project:developfrom
JaeGeon-SMU:GUIDE-62-어드민-기능-백엔드-작업

Conversation

@JaeGeon-SMU
Copy link
Copy Markdown
Collaborator

@JaeGeon-SMU JaeGeon-SMU commented Jan 17, 2024

error코드 API 문서에 작성되어 있는 기준 (첫째 자리 기준 ,둘째 자리 기준)을 바탕으로 폴더를 나누어 작성
-첫째 자리 기준으로 auth,user,event,vi,guide,admin,coach로 패키지를 나누었음
-둘째 자리 기준으로 dto,authorize,logic,resource 순으로 패키지를 나누었음
-아직 에러 코드가 없는 폴더의 경우에는 작성하지 않고 예시로 user관련 폴더만 만들어두었음
-메시지 국제화를 위한 MessageConfig 파일 생성 (yml파일에서 메시지 읽어 오는데 언어 별로 다른 메시지 보여주도록 설정)

vi 정보 입력 받기 구현
…/backend into GUIDE-62-어드민-기능-백엔드-작업

# Conflicts:
#	run/src/main/java/com/guide/run/global/security/user/CustomUserDetailsService.java
#	run/src/main/java/com/guide/run/user/controller/SignController.java
#	run/src/main/java/com/guide/run/user/dto/response/SignupResponse.java
#	run/src/main/java/com/guide/run/user/entity/Partner.java
#	run/src/main/java/com/guide/run/user/entity/User.java
#	run/src/main/java/com/guide/run/user/response/LoginResponse.java
#	run/src/main/java/com/guide/run/user/service/UserService.java
-메시지 국제화 라이브러리 의존성 추가
-국제화 관련 Service 및 Config 추가
-에러 코드 반환해 줄 dto FailResult 생성
-Security 설정 Config 파일로 통일
-API 문서를 바탕으로 만듦
-API 문서를 바탕으로 만듦
-API 문서를 바탕으로 만듦
-현재 에러코드 상에는 12xx번대가 없지만 미리 구현해 둠
-기존 ControllerAdvice들 파일명에 Advice 추가
-API 문서를 바탕으로 만듦
-NotExistUserException 에러만 캐치하던 것을 알맞은 에러로 수정
-API뮨서를 바탕으로 만듦
Copy link
Copy Markdown
Collaborator

@2ralumin 2ralumin left a comment

Choose a reason for hiding this comment

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

깃허브에 yml 파일은 안 올라와 있는데 혹시 카톡으로 따로 보내주실 수 있으면 보내주세요~
풀리퀘 날릴 때 resource는 안 올라가더라고요. 그리고 에러코드 만들 때 enum으로 코드 적기도 하던데 그건 없어도 되는 건가요?

@2ralumin 2ralumin merged commit c32ee49 into guiderun-project:develop Jan 19, 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