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/search history #71

Merged
merged 2 commits into from
Jul 30, 2024
Merged

Refactor/search history #71

merged 2 commits into from
Jul 30, 2024

Conversation

yeonjae02
Copy link
Member

@yeonjae02 yeonjae02 commented Jul 30, 2024

PULL REQUEST

#️⃣ 관련 이슈

📜 작업내용

  • 기존의 검색 기록 저장 api가 프론트 -> 백 데이터 전달에서 계속 오류가 발생하였습니다. 이를 해결하기 위해 새로운 api를 만들었습니다.

🔑 주요 변경사항

  • 기존의 history 코드 복사 후, 빈 충돌 방지를 위해 이름을 전부 기존 파일명+V2.java로 수정
  • 데이터 전달을 해결하기 위해 dto 분리 및 형태를 변경
  • 변경된 dto 형태에 맞게 서비스 계층, 컨트롤러 계층 코드 작성

🏞 스크린샷 (선택)

  • 내부 로직만 변화했기 때문에 결과 이미지는 기존 api와 동일합니다.

🗨️ 리뷰 요구사항(선택)

  • 기존 api도 테스트에서는 문제가 없었던 터라 프론트와의 빠른 확인을 위해 메인에 일단 바로 pr하겠습니다..
  • 기존 api도 삭제하진 않았고, 새로 만든 api를 /api/search-history-v2로 호출합니다.
  • v2 api로 문제가 해결되면 기존의 api를 삭제하고, 이름들을 변경할 계획입니다.

- 기존의 코드는 프 -> 백 데이터 전달 과정에서 계속 오류가 남.
- 이에 따라 백 측에서 데이터를 받아오는 코드 로직 자체를 변경한 v2를 작성함.
@yeonjae02 yeonjae02 self-assigned this Jul 30, 2024
@yeonjae02 yeonjae02 added bug Something isn't working feat 🌼연재🌼 labels Jul 30, 2024
@yeonjae02 yeonjae02 merged commit b79157e into main Jul 30, 2024
@yeonjae02 yeonjae02 mentioned this pull request Aug 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working feat 🌼연재🌼
Projects
None yet
Development

Successfully merging this pull request may close these issues.

검색 기록 저장 api v2 개발
1 participant