Skip to content

Conversation

@toychip
Copy link
Member

@toychip toychip commented Feb 4, 2024

Issue number and Link

이슈 번호

Summary

  1. 꿀팁공유해요 연관관계 매핑 오류 및 CRUD를 수정하였습니다.
  2. 꿀팁공유해요 수정 및 삭제시 쿼리가 n번 나가던 것을 1번으로 수정하였습니다.
  3. 꿀팁공유해요와 질문해요의 공통된 홈 화면에서 카테고리별로 10개씩 조회하는 로직을 추가하였습니다.
  4. CREATE, EDIT, DELETE시 단순한 성공 요청이 아닌 어떠한 타입의 어떠한 것의 몇번째가 어떻게 해결됐는지 알려주는 로직을 작성했습니다.
  5. 비용이 너무 많이 나와 cpu, memory를 낮췄습니다..

ToDo

  1. Member 엔티티가 생긴 후 Question, HoneyTip연관관계 매핑을 진행할 예정입니다.
  2. Member 엔티티가 생긴 후 좋아요, 스크랩 작업을 진행한 후에 오늘의 꿀팁 TOP5 로직을 진행할 예정입니다.
  3. 검색 기능은 함께해요, 소통해요와 병합한 후에 함께 같은 이슈에서 작업할 예정입니다.

PR Type

  • Feature
  • Bugfix
  • Refactoring
  • Documentation
  • Other

Other Information

ex) 후에 ~api 연결 필요

Common Type

- feat : 새로운 기능 구현
- add : feat 이외의 부수적인 코드, 파일, 라이브러리 추가
- chore : 패키지 구조, 함수 및 변수명 변경 등의 작은 작업
- fix : 버그 및 오류 해결
- del : 불필요한 코드, 파일, 주석 삭제
- docs : 명세서 작성
- refactor : 코드 리팩토링
- merge : 서로 다른 브랜치 간의 코드 병합
- setting : 프로젝트 기초 세팅 관련 작업

branch

Feature/{이슈 번호 x, feat 순서}-{작업 내용}

ex)
Feature/3-home-api
Feature/4-home-crud

@toychip toychip added 🐛 BugFix 버그 수정 ✨ Feature 새로운 기능 추가 🔀 PullRequest Branch 합병 임준형 labels Feb 4, 2024
@toychip toychip requested a review from LEEJaeHyeok97 February 4, 2024 15:53
@toychip toychip linked an issue Feb 4, 2024 that may be closed by this pull request
@toychip toychip self-assigned this Feb 4, 2024
Copy link
Contributor

@why-only-english why-only-english left a comment

Choose a reason for hiding this comment

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

준형님 honeytip, question 리팩토링 하느라 고생 많으셨습니다! 👍

Copy link
Contributor

Choose a reason for hiding this comment

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

message 객체 사용하여 service 코드 작성 시 어떤 장점이 있을까요?

Copy link
Member Author

Choose a reason for hiding this comment

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

저희가 생성, 수정, 삭제시에 엔티티의 Id만 응답했는데, 구체적으로 나타내는 것이 좋다고 판단해서 만들었습니다!

@psae0714
Copy link
Contributor

psae0714 commented Feb 5, 2024

준형님 수고하셨습니다. honeytip에 url을 따로 파일을 만드신 이유가 있으실 까요? 어떤 이점이 있나요?

@toychip
Copy link
Member Author

toychip commented Feb 5, 2024

준형님 수고하셨습니다. honeytip에 url을 따로 파일을 만드신 이유가 있으실 까요? 어떤 이점이 있나요?

저희 설계에 있는, 대화를 할 수 있는 오픈채팅 uri의 uri입니다!

@toychip toychip merged commit 2224509 into develop Feb 5, 2024
@toychip toychip deleted the Feat/4-HoneyTip-Question-common-api branch October 17, 2024 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 BugFix 버그 수정 ✨ Feature 새로운 기능 추가 🔀 PullRequest Branch 합병 임준형

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: '꿀팁 공유해요' 와 '질문해요' 의 공통 메인 Api

4 participants