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

사장님 회원가입, 식당 등록 #8

Closed
bgpark82 opened this issue Jul 20, 2022 · 0 comments
Closed

사장님 회원가입, 식당 등록 #8

bgpark82 opened this issue Jul 20, 2022 · 0 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@bgpark82
Copy link
Contributor

No description provided.

@bgpark82 bgpark82 added the enhancement New feature or request label Jul 20, 2022
@bgpark82 bgpark82 self-assigned this Jul 20, 2022
bgpark82 pushed a commit that referenced this issue Jul 20, 2022
bgpark82 pushed a commit that referenced this issue Jul 20, 2022
bgpark82 pushed a commit that referenced this issue Jul 20, 2022
@taekyun95 taekyun95 added this to the 6주 milestone Jul 20, 2022
bgpark82 pushed a commit that referenced this issue Jul 24, 2022
bgpark82 pushed a commit that referenced this issue Jul 24, 2022
bgpark82 pushed a commit that referenced this issue Jul 24, 2022
프론트엔드에서 비밀번호 확인 처리
bgpark82 pushed a commit that referenced this issue Jul 24, 2022
- 예외가 계속 추가되면 @ExceptionHandler, @RestControllerAdvisor으로 변경 예정
- 참고: https://spring.io/blog/2013/11/01/exception-handling-in-spring-mvc
bgpark82 added a commit that referenced this issue Jul 25, 2022
* #8 라이브러리 추가

* #8 git ignore 추가 (application-local.yml)

* #8 사장님 회원가입

* #8 식당 등록

* #8 사장님 회원가입 url 변경

* #8 POST 메소드 응답 상태 변경

- 204 -> 200

* #8 비밀번호 확인 로직 제거

프론트엔드에서 비밀번호 확인 처리

* #8 커스텀 예외 추가

- 예외가 계속 추가되면 @ExceptionHandler, @RestControllerAdvisor으로 변경 예정
- 참고: https://spring.io/blog/2013/11/01/exception-handling-in-spring-mvc

* #8 사장님 회원가입 에지 케이스 테스트 추가

* #8 사장님 회원가입 http 요청 추가

Co-authored-by: peter-alethio <peter@alethio.io>
taekyun95 added a commit that referenced this issue Aug 19, 2022
* 사장님 회원가입, 식당 등록 (#9)

* #8 라이브러리 추가

* #8 git ignore 추가 (application-local.yml)

* #8 사장님 회원가입

* #8 식당 등록

* #8 사장님 회원가입 url 변경

* #8 POST 메소드 응답 상태 변경

- 204 -> 200

* #8 비밀번호 확인 로직 제거

프론트엔드에서 비밀번호 확인 처리

* #8 커스텀 예외 추가

- 예외가 계속 추가되면 @ExceptionHandler, @RestControllerAdvisor으로 변경 예정
- 참고: https://spring.io/blog/2013/11/01/exception-handling-in-spring-mvc

* #8 사장님 회원가입 에지 케이스 테스트 추가

* #8 사장님 회원가입 http 요청 추가

Co-authored-by: peter-alethio <peter@alethio.io>

* Feature/tgk 4 user curd (#7)

* #4 라이브러리 추가

spring-boot-starter-validation: 데이터 유효검 검사용
spring-boot-devtools: 개발편의성
mybatis-spring-boot-starter-test: Mybatis Test용
mapstruct: Object Mappering용

* #4 설정추가 (DB, Mybatis)

Mybatis: 
  mapper-locations: 매퍼 위치 설정
  map-underscore-to-camel-case: 테이블 컬럼명을 스네이크 케이스를 커멀케이스로 매핑해주는 설정

* #4 사용자 회원가입

* #4 UserObjectMapper 오브젝트 매핑 설정 추가

id => 무시
createDate, updateDate => LocalDateTime.now()

* #4 테스트 메소드명 한글로 변경

영어에서 한글로 변경

* #4 기능: 세션 로그인 추가

* #4 오타 수정

* #4�기능: 사용자 세션 로그아웃 추가

* 코드 리뷰 반영

1. * import 풀어 사용하기
2. static String을 한 곳에서 모이기
3. 구분의 의미 없는 공백 제거
4. DTO String 속성 길이 제한
5. DateTime ​사용하는 변수에 이름을 정확히 표현
6. gradle 의존성 namespace를 그룹핑 + 사전 순 정렬

* #4 코드 리뷰 반영

1. * import 풀어 사용하기
2. static String을 한 곳에서 모이기
3. 구분의 의미 없는 공백 제거
4. DTO String 속성 길이 제한
5. DateTime ​사용하는 변수에 이름을 정확히 표현
6. gradle 의존성 namespace를 그룹핑 + 사전 순 정렬

* #4 JSON 방식 스프링 시큐리티 적용

기타
RegisterDto: @Setter 제거
UserController: 로그인, 로그아웃 제거
UserObjectMapper: password 인코딩 매핑 추가
LoginDtoValidator: 제거
JUnit Assertion => AssertJ

* 세션 저장소 변경: 로컬 저장소에서 레디스로

* 변경 LocalDateTime에서 ZonedDateTime으로

* 변경 클래스명: UserContext에서 RestUserDetails로

* #4 로그인, 로그아웃, 회원가입 URL 규칙 적용

* #4 시큐리티 클래스 명칭 수정

RestXXX => GeneralXXX

* #4 Validation 테스트 추가

* #4 AuthenticationProcessingFilter에 validation추가

* 공백 정리

* #13 사장님 로그인 추가 (#17)

* #13 사장님 로그인 추가

* #13 OwnerAuthentication...과 GeneralAuthentication...의 공통된 부분을 추상화 처리

* 리팩토링

불필요한 파일 삭제, 패키지 이동

* #11 메뉴 추가

* 추가: 메뉴 추가 기능
* 추가: 가게에 가게ID가 있는지 조회하는 기능
* 추가: 가게에 가게ID와 사장짐 ID가 있는지 조회하는 기능

* #11 코드 리뷰 반영

- ExceptionHandler사용
- Constants OWNER제거
- SecurityConfig "/store/*/menus" -> "/stores/*/menus"
- 과거, 사장님의가게 검증을 StoreService로 이동

Co-authored-by: Byeonggil Park <bgpark82@gmail.com>
Co-authored-by: peter-alethio <peter@alethio.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants