Skip to content

fix: 대학 정보 전체조회 API 엔드포인트 보정#439

Merged
manNomi merged 3 commits intomainfrom
fix/univ-api-endpoint
Feb 16, 2026
Merged

fix: 대학 정보 전체조회 API 엔드포인트 보정#439
manNomi merged 3 commits intomainfrom
fix/univ-api-endpoint

Conversation

@manNomi
Copy link
Contributor

@manNomi manNomi commented Feb 16, 2026

Summary

  • 대학 목록 전체 조회 시 사용하던 API 경로를 실제 동작하는 전체 조회 endpoint로 수정했습니다.
  • main 기준에서 단일 커밋만 포함되도록 브랜치를 정리해 불필요한 변경이 PR에 섞이지 않게 했습니다.
  • 로컬에서 web ci:checkweb build를 모두 통과해 pre-push 실패 원인이던 빌드 오류가 재발하지 않음을 확인했습니다.

@vercel
Copy link

vercel bot commented Feb 16, 2026

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

Project Deployment Actions Updated (UTC)
solid-connection-web Ready Ready Preview, Comment Feb 16, 2026 11:15am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
solid-connect-web-admin Skipped Skipped Feb 16, 2026 11:15am

@github-actions github-actions bot added the web label Feb 16, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 16, 2026

Warning

Rate limit exceeded

@manNomi has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 4 minutes and 28 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

🔍 Walkthrough

이번 변경은 대학 검색 관련 API 엔드포인트의 경로를 수정한 작은 규모의 업데이트입니다.

  1. 대학 정보 검색 엔드포인트 경로 변경
    • /univ-apply-infos/search/filter에서 /univ-apply-infos/search/text?value=로 수정
    • getSearchUniversitiesAllRegions 함수의 요청 경로만 변경되었으며, 데이터 처리 로직이나 에러 핸들링은 그대로 유지됨

📊 Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

👥 Suggested reviewers

  • wibaek
🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경사항의 핵심을 명확하게 요약하고 있습니다. 대학 정보 조회 API 엔드포인트 수정이라는 주요 변경을 정확히 반영하고 있습니다.
Description check ✅ Passed PR 설명이 작업 내용, 브랜치 정리, 검증 완료를 포함하여 필수 정보를 충분히 담고 있습니다. 템플릿의 핵심 섹션이 다 포함되어 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/univ-api-endpoint

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 and usage tips.

@vercel vercel bot temporarily deployed to Preview – solid-connect-web-admin February 16, 2026 11:11 Inactive
@manNomi manNomi merged commit b6c9c01 into main Feb 16, 2026
10 checks passed
manNomi added a commit that referenced this pull request Feb 22, 2026
* fix: 대학 정보 조회 endpoint를 전체 조회 경로로 수정

* fix: 대학 상세 목록 ISR 시간을 Infinity로 조정

* fix: 대학 상세 목록 ISR 재검증 주기를 6개월로 설정
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