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

필터링 구현, 토큰 에러 응답 양식 변경 #101

Merged
merged 5 commits into from
Oct 15, 2023

Conversation

seokwns
Copy link
Contributor

@seokwns seokwns commented Oct 15, 2023

작업 내용

  • 포트폴리오 필터링 구현 (쿼리 스트링으로 name=가나다&location=부산 이런 식으로 보내면 됩니다)
  • 토큰 관련 에러 응답 양식 변경

주의 사항(Optional)

  • 테스트를 꼼꼼히 진행하지 않아서 .. 오류 발생하면 말씀해주세요 ㅜㅜ
  • 테스트 코드는 다음 주에 작성해서 올려두겠습니다. 먼저 해주셔도 됩니다 !! 다음주에 안올라오면 제가 작성해서 올리도록 하겠습니다

이슈 번호

close #99
close #100

PR을 두개로 분리해야 했는데 .... 요즘 자꾸 깜빡해서 같이 보냅니다 죄송합니다 ㅜㅜ
🌟 벌써 이슈가 100개를 돌파했습니다 @~@

- 플래너의 이름과 지역을 이용한 포트폴리오 필터링 구현
- 토큰 에러 발생 시 에러 항목의 포맷 변경
- 필터링 조건이 null 문자열이라면 전체 포트폴리오를 검색하도록 수정
- 테스트 에러 수정
@seokwns seokwns added enhancement New feature or request Done 진행 완료 portfolio 포트폴리오 관련 기능 refactor labels Oct 15, 2023
@seokwns seokwns self-assigned this Oct 15, 2023
- 테스트 에러 수정
Copy link
Member

@1jeongg 1jeongg left a comment

Choose a reason for hiding this comment

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

TokenException 도 생성하시고 필터링 부분도 완전 깔끔하게 구현해주셨네요 ㅠㅠ 고생하셨어요👍👍
아 그리고 저기 chatId가 1인건 sql에서 match 테이블은 id 1번을 주석처리해 둬서 그래요,, auto increment가 제대로 안 돼서 임의로 저렇게 처리했는데 영채님이 알려주셔서 나중에 수정해볼게요!!

나중에 시간 남으면 location은 constant하니까 enum type으로 바꿔도 좋을 것 같아요! (하지만 기능 구현이랑 버그 잡아야 해서 나중으로 마뤄야할 것 같네요 ㅠㅠ)

@1jeongg 1jeongg merged commit ca3faf3 into Step3-kakao-tech-campus:weekly Oct 15, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Done 진행 완료 enhancement New feature or request portfolio 포트폴리오 관련 기능 refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

토큰 관련 오류 응답 양식 변경 포트폴리오 필터링 기능 구현
2 participants