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

Refactor/mentor profile update #131

Conversation

koreanddinghwan
Copy link
Contributor

- mentorProfile social link not null
- 프론트측에서 optional dto를 사용하지 않는것으로 확인되어 optional
  해제했습니다.
- activation api를 새로 생성했습니다.

- dto에서 null check를하기때문에 controller에서 하지않습니다.
- activation dto 별도로 생성.
- dto nullable 해제. socialLink는 optional로 null인경우 regex 확인X
- null이 아닌경우 regex에서 체크함.
- update할때 프로필 숨겨야할때 validation추가. socialLink의 경우,
  Null을 체크
@koreanddinghwan
Copy link
Contributor Author

프론트 PR에 RTK/User.ts에서 isHide 사용하는 api url 변경 커밋했고,
로컬에서 작동하는것 테스트했습니다.

Copy link
Member

@JuneParkCode JuneParkCode left a comment

Choose a reason for hiding this comment

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

늦은 시간에 고생 많으셨습니다..
직접 테스트는 못했고 코드상으로는 문제 없어보입니다.

.github/workflows/deploy-api-docs.yml Show resolved Hide resolved
@koreanddinghwan koreanddinghwan merged commit 156b0da into manito42:develop Oct 5, 2023
1 check passed
@koreanddinghwan koreanddinghwan deleted the refactor/mentorProfile_update branch October 23, 2023 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants