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: remove optional condition to counselor card totalConsult (#348) #359

Merged
merged 1 commit into from
Jul 21, 2024

Conversation

kyuhho
Copy link
Contributor

@kyuhho kyuhho commented Jul 20, 2024

Checklist


  • 올바른 브랜치에 PR을 보내도록 설정하였나요?
  • PR의 라벨을 올바르게 달았나요?

Description


기존에 찜 목록 리스트에 `totalConsult`(상담 횟수)가 없어서 optional 처리하였는데, api에 추가되어 해당 사항 반영하였습니다.

Related Issues


#348

To Reveiwer


확인 부탁드립니다!

@kyuhho kyuhho added API API 관련 Feature 기능 개발 Priority-2 priority 2 Stage-2/code-review-request work done and CI passed labels Jul 20, 2024
@kyuhho kyuhho self-assigned this Jul 20, 2024
Copy link

netlify bot commented Jul 20, 2024

Deploy Preview for candid-semolina-d0db42 ready!

Name Link
🔨 Latest commit c7590d0
🔍 Latest deploy log https://app.netlify.com/sites/candid-semolina-d0db42/deploys/669b997fde7b100008110e8d
😎 Deploy Preview https://deploy-preview-359--candid-semolina-d0db42.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@rmdnps10 rmdnps10 left a comment

Choose a reason for hiding this comment

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

확인했습니다!

@rmdnps10 rmdnps10 merged commit 9406aa7 into dev Jul 21, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API API 관련 Feature 기능 개발 Priority-2 priority 2 Stage-2/code-review-request work done and CI passed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants