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

feat, setting: 회원가입 Validation, 응답폼 작성, 프로젝트 세팅 수정 #43

Merged
merged 11 commits into from
Nov 15, 2022

Conversation

soomanbaek
Copy link
Member

Issues

🤷‍♂️ Description

세팅 수정
(실수로 이 브랜치에서 작업했어요 ㅠ)

  • 폴더 이름 변경 (common -> shared)
  • 폴더 경로 변경 (/user/auth -> /auth)
  • ts.config에 api 경로 추가

구현

  • 응답포맷 작성
  • 회원가입시
    • 비밀번호 해싱
    • 아이디 중복 시 예외처리
    • DB에 저장

📒 Remarks

굉장히 광범위하게 수정된 것이 많아서 나영님 PR 하실 때 conflict 많이 날 것 같아요 🙇‍♂️.. 같이 수정해봐요~

@soomanbaek soomanbaek added 🌐 server 서버 작업 ✨ feature 기능 추가 🔧 project-settings 프로젝트 세팅 labels Nov 15, 2022
@soomanbaek soomanbaek added this to the ⛰️ Week2 milestone Nov 15, 2022
@soomanbaek soomanbaek self-assigned this Nov 15, 2022
@NaayoungKwon
Copy link
Collaborator

  • response는 데이터 반환이 필수가 아닌 경우 message가 필요없을 것 같다고 FE와 협의되어 삭제 부탁드립니다.

@NaayoungKwon NaayoungKwon merged commit f58e103 into dev-be Nov 15, 2022
@soomanbaek soomanbaek deleted the feature/auth branch November 15, 2022 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feature 기능 추가 🔧 project-settings 프로젝트 세팅 🌐 server 서버 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants