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

ADD :: 사람 추가 api (#63) #73

Merged
merged 34 commits into from
Apr 2, 2023
Merged

ADD :: 사람 추가 api (#63) #73

merged 34 commits into from
Apr 2, 2023

Conversation

Yunyeong-Ko
Copy link
Contributor

📄 Summary

그룹에 학생 추가


🔨 Tasks

  • 그룹에 학생 추가 구현

🙋🏻 More

close #63

gimhanul and others added 21 commits March 24, 2023 18:11
- querydsl 사용하기 위한 custom repository 추가
- authority 동적쿼리
- 현재 필수값이 아닌데 응답을 하지 않은 경우 Null로 저장되어 공간이 낭비되고 있었음
- 해당 경우에 값을 저장하지 않도록 수정함
- question에 type 추가
- reply에 questino id추가
- null check를 하기 위함
- query reply 할 때 필수가 아닌 값 null처리해줌
- 설명 size validation을 주지 않아서 서버 에러 오류로 뜨는 현상 발생
- size validation 추가
- 기존 200자 -> 500자로 수정
- 사람 타입 추가
- 사람 쿼리할 때 이름으로 쿼리하도록 변경
- searh response 변경
- search user에 통합
Copy link
Member

@gimhanul gimhanul left a comment

Choose a reason for hiding this comment

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

Security Config에서 권한 Teacher로 제한해주세용

@Yunyeong-Ko Yunyeong-Ko force-pushed the feat/#63-add branch 2 times, most recently from 146be92 to ea179ca Compare March 31, 2023 06:50
@Yunyeong-Ko Yunyeong-Ko reopened this Mar 31, 2023
@Yunyeong-Ko Yunyeong-Ko merged commit a5220c4 into feat/group Apr 2, 2023
@Yunyeong-Ko Yunyeong-Ko deleted the feat/#63-add branch April 2, 2023 11:33
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