Skip to content

fix: 저장 동아리 후기 표시 및 운영진 검색 기록 분리 - #182

Merged
dennis0405 merged 1 commit into
developfrom
patch/club-history-reviews
Oct 6, 2026
Merged

dennis0405 merged 1 commit into
developfrom
patch/club-history-reviews

Conversation

@dennis0405

@dennis0405 dennis0405 commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator

작업 내용

  • 저장 탭의 동아리 카드에 실제 후기 수와 후기 키워드를 반환합니다.
  • 운영진 등록의 동아리 검색은 탐색 탭의 최근 검색어에 저장하지 않습니다.
  • 일반 탐색 검색과 구버전 앱은 기존처럼 최근 검색어를 저장합니다.

변경 이유

구현 및 호환성

  • 저장 목록은 기존 getClubReviews 일괄 집계 함수를 재사용합니다. 공개/삭제 필터와 응답 순서는 유지합니다.
  • 검색 API에 선택적 record_recent_search=true|false를 추가했습니다. 생략 시 true이며, false인 경우 검색 결과/인증은 유지하고 기록 저장만 생략합니다.
  • 운영진 등록 화면만 false를 전달합니다. 잘못된 값/중복 값은 기존 검색 필터 검증과 동일하게 400을 반환합니다.
  • OpenAPI 검색 query 스키마에 새 옵션과 기본 동작을 문서화했습니다.
  • DB 스키마 변경, 기존 검색 기록 삭제, 로그인/권한 이전/네이티브 시작 화면 변경은 없습니다.
  • 저장 후기 수정은 서버 배포로 반영됩니다. 운영진 등록 검색 분리는 서버와 새 앱이 모두 반영되어야 합니다.

테스트 및 리뷰

  • 회귀 테스트: 저장 후기 2개, 검색 API 8개, 모바일 검색 요청 3개 추가.
  • 수정 전 회귀 테스트 실패 → 수정 후 통과 확인. 검색 저장 조건을 반대로 바꾼 변이 검증에서도 5개 테스트가 실패하는 것을 확인한 뒤 원복했습니다.
  • pnpm verify 통과: 웹 106 tests 통과 / PostgreSQL 통합 1개 로컬 환경 미설정으로 skip, 모바일 30 suites / 170 tests 통과.
  • 웹 production build, Android·iOS JS bundle, Biome/TypeScript, git diff --check 통과.
  • 데이터 접근/인증 경계, 빈 저장 목록, 후기 없는 동아리, guest/member 및 기본값 호환성을 검토했습니다.
  • 이 수정의 실기기 QA는 아직 미실시입니다. 기존 #177의 양 플랫폼 QA 결과와 구분합니다.
  • PR CI 4/4 통과: https://github.com/wafflestudio/allclear/actions/runs/37411538980 . PostgreSQL 통합 테스트를 포함한 웹 107 tests 및 모바일 170 tests 통과.
  • 최종 production 의존성 audit는 critical 0 / high 27입니다. 기존 #181의 최소 보안 패치 범위를 유지했으며 잔여 High 해소를 의미하지 않습니다.

배포 / 롤백

  • patch/club-history-reviews → develop → main(release: 관리자 권한 이전 및 모바일 QA 개선 (develop → main) #179) 순서로 반영합니다.
  • DB migration이 없으며 장애 시 이 커밋 revert 또는 이전 웹 이미지/앱으로 복귀할 수 있습니다. 기존 앱과 서버 간 API 호환성을 유지합니다.
  • main 승인 리뷰 및 최종 출시 확인은 이 patch CI와 별도입니다.

@dennis0405
dennis0405 merged commit c718bb3 into develop Oct 6, 2026
5 checks passed
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