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

[ST-662] 리뷰 관련 엔드포인트 추가 #83

Merged
merged 5 commits into from
Oct 9, 2023
Merged

[ST-662] 리뷰 관련 엔드포인트 추가 #83

merged 5 commits into from
Oct 9, 2023

Conversation

w8385
Copy link
Member

@w8385 w8385 commented Oct 8, 2023

업데이트 유형

  • Hot Fix
  • Release
  • Develop
  • Others

업데이트 개요

  • 학생 - 과외에 별점 매기기
  • 선생님 - 프로필에서 별점 확인
  • 사용자 프로필 내 과외 히스토리 만들기

업데이트 요약

  • /user/history/{userId}
    • 해당 사용자의 튜터링 히스토리 반환
  • /teacher/rating/{teacherId}
    • 해당 선생님의 레이팅 반환
    • 다른 프로필을 호출하는 엔드포인트에도 적용됨

예시: /user/profile/5cb4a32c-87f7-45f5-8906-07c960641f0f

{
  "message": "사용자 프로필을 성공적으로 가져왔습니다.",
  "success": true,
  "data": {
    "id": "5cb4a32c-87f7-45f5-8906-07c960641f0f",
    "name": "앨런 튜우링",
    "bio": "lets study!",
    "profileImage": "https://short-tutoring.s3.ap-northeast-2.amazonaws.com/default/profile-img/ic_profile_dog.png",
    "role": "teacher",
    "school": {
      "name": "한양대학교",
      "department": "수학교육과"
    },
    "followers": [
      "b3c49107-86ac-4e30-95c4-ba02b7a2c16c"
    ],
    "followingCount": 0,
    "rating": 4.5
  }
}
  • 잡버그들 수정
  • Question 리스트 내려줄 때 시간순 정렬

해결한 문제

미해결 문제

테스트

  • 유닛 테스트
  • 빌드 테스트 (통합 테스트)
  • 기타 유효성 테스트

수정 사항 진단

  • 코드가 이 프로젝트의 스타일 지침을 따릅니다.
  • 코드에 대한 자체 리뷰를 수행했습니다.
  • 변경 내역에 대해 주석을 작성했습니다.
  • 해당 PR에 대한 문서를 변경했습니다.
  • 기능, 수정 사항, 이슈에 대한 테스트 코드를 추가했습니다.
  • 변경 내용이 로컬 개발환경에서의 테스트를 통과했습니다.

@w8385 w8385 requested a review from seongyunlee October 8, 2023 15:11
@w8385 w8385 changed the title Feat/review [ST-662] 리뷰 관련 엔드포인트 추가 Oct 9, 2023
@notion-workspace
Copy link

@w8385 w8385 merged commit 5acb4ee into dev Oct 9, 2023
3 checks passed
@w8385 w8385 deleted the feat/review branch October 9, 2023 04:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant