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.

Sign up for GitHub

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

Jump to bottom

Doc: README.md 추가 #39

Merged
merged 5 commits into from
Oct 27, 2023
Merged

Doc: README.md 추가 #39

merged 5 commits into from
Oct 27, 2023

Conversation

kshshkim
Copy link
Contributor

  • 커스텀 로고
  • 사용 기술 목록
  • Content 목록 조회 기능에 대해 고민한 것들과 마주친 문제, 해결방안 등.

Ref: #8

@kshshkim
Doc: README.md 추가
08b3868 
- 커스텀 로고
- 사용 기술 목록
- Content 목록 조회 기능에 대해 고민한 것들과 마주친 문제, 해결방안 등.
@kshshkim
Doc: minor fix
3770b85 
오탈자 수정
@kshshkim
Doc: minor fix
103c736 
오탈자 수정
@kshshkim
Doc: minor fix
31dec63 
오탈자 수정, 스타일 수정
@kshshkim
Doc: minor fix
104776f 
어휘 수정
@kshshkim kshshkim merged commit 43ed8a6 into master Oct 27, 2023
@kshshkim kshshkim deleted the readme branch October 27, 2023 12:08
@kshshkim kshshkim restored the readme branch October 27, 2023 12:16
kshshkim pushed a commit that referenced this pull request Oct 29, 2023
@jaekkang
Feat: Member controller 가입 승인(#60)
0406764 
* Init: Create member repository

멤버 데이터를 저장하기 위한 저장소를 만들었습니다.
기본적인 영속성 추가를 위한
save()와 단건 데이터 검색과 전체 조회 메서드를 추가했습니다.

* Refactor: Change MemberRepository type

MemberRepository의 형식을 JPA Repository를 상속받는 형식으로 수정하였습니다.

* Feat: Add method for findByName

name을 기준으로 검색을 하는 메서드를 추가했습니다.

* Refactor: Change logic in create method

생성 메서드를 Builder 사용하는 방식으로 수정하였습니다.

* Style: Change method name

잘못 작성한 메서드의 이름을 변경하였습니다.
findByName -> findByusername

* Test: Add MemberRepository Test

작성한 Member Repository Test 추가했습니다.

간단히
- 빌더통해 생성 확인
- 저장한 데이터 불러오기
- 잘못된 색인시 에러 나는지

* Feat: Add hasher

Hasher 부분이 사용되지 않은 것을 확인해서 추가했습니다.

* Init: Create DTO for signup method

회원가입시 사용될 DTO를 작성했습니다.

* Feat: Add signupMember

가입 승인을 신청하는 url을 맵핑했습니다.

* Feat: Remove validate

validator가 잘못된 곳에서 사용되고 있어서 삭제했습니다.

* Feat: Add optional for empty data

Repository에서 서치시 데이터가 없을 경우를 위해 Optional로 감쌋습니다.

* Test: remove default method

빌더를 통한 객체 생성 테스트를 제외하고 jpaRepository에서 자동 생성되는 메서드의 테스트는 삭제했습니다.

* Style: Change method name

- validatePassword를 의미적으로 좀 더 맞는
verifyPassword로 변경되었습니다.

- 메서드에 대한 설명도 추가되었습니다.

* Test: Change test

Test문이 줄어 빌더 생성관련 밖에 남지 않아 MemberTest로 변경되었습니다.

* Doc: README.md 추가 (#39)

* Doc: README.md 추가

- 커스텀 로고
- 사용 기술 목록
- Content 목록 조회 기능에 대해 고민한 것들과 마주친 문제, 해결방안 등.

* Refactor: add return value in signupMember()

signupMember 메서드의 리턴 밸류가 void였는데  response status 200을 주는 것으로 변경했습니다.

* Feat: Add fields in SignupMemberDTO

가입 승인 로직에서 사용될 DTO에 적절한 필드를 추가하였습니다.

* Init: Add tmp member sevice

Contoller의 동작을 테스트 하기 위한 임시 service를 작성하였습니다.

* Comment: Add comment for signupMember method

signupMember 메서드를 위한 주석을 추가했습니다.

* Test: Add test for member controller

멤버 Controller의 동작을 확인하기 위한 테스트를 작성했습니다.

* Refactor: Remove response wildcard

ResponseEntity를 와일드카드로 반환하는 부분을 void로 변경

* Refactor: Remove SignupMemberDTO

목적이 같은 DTO를 삭제 및 Confirm객체로 교체하였습니다.

* Refactor: Remove tmp memberServiceImpl

임시로 넣어서 사용했던, MemberServiceImpl을 삭제하고, Controller에서도 인터페이스에 종속되도록 변경하였습니다.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kshshkim