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

시니또용 콜백 조회 API 및 기능 수정 #130

Merged
merged 5 commits into from
Nov 1, 2024
Merged

Conversation

JYN523
Copy link
Collaborator

@JYN523 JYN523 commented Oct 30, 2024

#️⃣ 연관된 이슈

ex) #이슈번호, - #이슈번호

#101

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요.(이미지 첨부 가능)

  • 콜백 단건 조회 API 수정사항 반영 (전화번호, 진행상태에 추가)
  • 가이드라인 조회 API 수정사항 반영
  • 코드리뷰 수정사항 반영 (type 이름 변경)

스크린샷 (선택)

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

API 수정사항을 확인하고, 콜백/가이드라인 각각 다시 수정 요청을 해서 반영된 걸 확인한다고 늦어졌습니다...
현재 가이드라인 API 수정사항은 백엔드 배포 서버에 반영되어있어 정상적으로 실행되고, 콜백 API 수정사항은 아직 백엔드 배포 서버에 반영되지 않은 걸로 알고 있습니다. 콜백 API는 성훈님 개인서버로 정상적으로 동작되는것을 확인했습니다.

⏰ 현재 버그

✏ Git Close

close #이슈번호

close #101

@JYN523 JYN523 added ✨ Feature 새로운 기능 추가 및 구현하는 경우 📡 API 비동기 통신 코드를 짜는 경우, 백엔드와의 통신하는 경우 labels Oct 30, 2024
@JYN523 JYN523 requested review from Diwoni and Dobbymin October 30, 2024 11:27
@JYN523 JYN523 self-assigned this Oct 30, 2024
@JYN523 JYN523 linked an issue Oct 30, 2024 that may be closed by this pull request
2 tasks
Copy link
Collaborator

@Diwoni Diwoni left a comment

Choose a reason for hiding this comment

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

TODO 사항 잘 반영되었네요! 고생하셨습니다!

@@ -1,6 +1,6 @@
import { CallbackResponse } from '@/shared/types';

export type CallbacksResponse = {
export type CallbackListResponse = {
Copy link
Collaborator

Choose a reason for hiding this comment

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

타입 명도 확실히 이게 더 보기 좋네요

@Dobbymin Dobbymin merged commit c381661 into Weekly Nov 1, 2024
1 check passed
@Dobbymin Dobbymin deleted the Feat/issue-#101 branch November 1, 2024 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📡 API 비동기 통신 코드를 짜는 경우, 백엔드와의 통신하는 경우 ✨ Feature 새로운 기능 추가 및 구현하는 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

시니또용 콜백 조회 API 및 기능 수정
3 participants