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

feat: 내 약통 관련 api 연동 #122

Merged
merged 9 commits into from
Feb 27, 2025
Merged

feat: 내 약통 관련 api 연동 #122

merged 9 commits into from
Feb 27, 2025

Conversation

DongjaJ
Copy link
Collaborator

@DongjaJ DongjaJ commented Feb 27, 2025

이슈 번호

작업한 목록을 작성해 주세요

  • 상세 페이지에서 약통에 추가, 삭제하는 api 연동
  • 내 약통 페이지에서 약 목록 조회 api 연동 -> ui가 구현이 안돼있습니다
  • 내 약통 관리 페이지 약 목록 조회 api 연동
  • 내 약통 추가 페이지 검색 및 내 약통에 추가 api 연동

스크린샷

pr 포인트나 궁금한 점을 작성해 주세요

  • 약통에서 삭제하는 건 백엔드에서 다건 삭제 api가 나오면 연동하려고 합니다
  • 키워드 자동완성도 백엔드가 완성되면 연결하려고 합니다
  • 약을 조회할때 태그들이 많을 경우 어떻게 보여줄 지 논의가 필요할 것 같습니다
  • 같은 종류의 약을 내 약통에 추가했을 때 중복으로 보이는 데 논의가 필요할 것 같습니다

연관된 issue: #121

@DongjaJ DongjaJ requested a review from ptq124 as a code owner February 27, 2025 13:23
@DongjaJ DongjaJ linked an issue Feb 27, 2025 that may be closed by this pull request
3 tasks
Copy link

📚 TypeDoc 문서가 배포되었습니다:

Copy link

Copy link
Collaborator

@ptq124 ptq124 left a comment

Choose a reason for hiding this comment

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

약을 조회할때 태그들이 많을 경우 어떻게 보여줄 지 논의가 필요할 것 같습니다

  • 다 보이게 하는게 애매할거 같네요. 이건 인덱스 0~2번정도까지 보이면 좋을거 같아용

같은 종류의 약을 내 약통에 추가했을 때 중복으로 보이는 데 논의가 필요할 것 같습니다

  • 이 부분은 백단에서 하는게 쉬울거 같아용. distinct나 내 약통 불러 오기 api 로직 조금만 바꿔 이용하면 될거 같아요 이건 백엔드 분께 문의해봅시다!

@DongjaJ DongjaJ merged commit a9984c6 into main Feb 27, 2025
9 checks passed
@DongjaJ DongjaJ deleted the feat-121 branch February 27, 2025 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: 내 약통 페이지 api 연동
2 participants