Skip to content

fix: 멘토 및 채팅 목록 페이지 번호 수정 - #634

Merged
manNomi merged 1 commit into
mainfrom
fix/mentor-chat-pagination
Sep 1, 2026
Merged

fix: 멘토 및 채팅 목록 페이지 번호 수정#634
manNomi merged 1 commit into
mainfrom
fix/mentor-chat-pagination

Conversation

@manNomi

@manNomi manNomi commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

관련 이슈

  • 없음

작업 내용

  • 서버의 1 기반 페이지 규칙에 맞춰 멘토 신청자 목록의 첫 요청을 page=1로 수정했습니다.
  • 멘티 신청 목록과 멘토 검색 목록의 조회 및 프리페치도 같은 페이지 규칙으로 통일했습니다.
  • 채팅 기록의 첫 요청과 API 기본값을 page=1로 수정해 채팅방 재진입 시 저장된 메시지를 다시 불러오도록 했습니다.

특이 사항

  • 다음 페이지는 기존과 같이 서버 응답의 nextPageNumber를 그대로 사용합니다.
  • 로그인 계정을 이용한 실제 화면 검증은 진행하지 못했습니다.

검증

  • pnpm exec biome check src/apis/chat/api.ts src/apis/chat/getChatMessages.ts src/apis/mentor/getAppliedMentorings.ts src/apis/mentor/getMentorList.ts src/apis/mentor/getReceivedMentorings.ts
  • pnpm exec tsc --noEmit -p apps/web/tsconfig.ci.json --incremental false

리뷰 요구사항 (선택)

  • 멘토 및 채팅 API가 모두 서버의 1 기반 페이지 계약과 일치하는지 확인 부탁드립니다.

@manNomi
manNomi requested a review from wibaek as a code owner September 1, 2026 14:39
@vercel

vercel Bot commented Sep 1, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
solid-connect-university-web Ready Ready Preview Sep 1, 2026 2:42pm UTC
solid-connection-web Ready Ready Preview Sep 1, 2026 2:42pm UTC
1 Skipped Deployment
Project Deployment Actions Updated
solid-connect-web-admin Skipped Skipped Sep 1, 2026 2:42pm UTC

@coderabbitai

coderabbitai Bot commented Sep 1, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 45 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Team

Run ID: 99ede4c9-c0ce-419b-aa2b-1b98d0f7f0b7

📥 Commits

Reviewing files that changed from the base of the PR and between 03e914b and 052adde.

📒 Files selected for processing (5)
  • apps/web/src/apis/chat/api.ts
  • apps/web/src/apis/chat/getChatMessages.ts
  • apps/web/src/apis/mentor/getAppliedMentorings.ts
  • apps/web/src/apis/mentor/getMentorList.ts
  • apps/web/src/apis/mentor/getReceivedMentorings.ts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 1, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-01T14:41:44.952427Z 052adde PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant