refactor: v1 API 및 레거시 DB 필드 제거 - #147
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
작업 내용
device기능 제거deviceEntity 및 datasource 등록 제거/api/v2/users/me/devicesAPI 제거updateDevice및 관련 OpenAPI 스키마 제거POST /api/v2/managers/me/clubs제거club_manager_register_request관련 코드 제거club_manager_request흐름으로 일원화collegeMajor기반으로 일원화allclear_user.college,major의존성 제거account.passwordallclear_user의college,major,grade, university 관련 컬럼user의 개인정보·약관 동의·service컬럼club의 기존 상세 정보 및 인기 동아리 관련 컬럼user_club_review.rating,contentapps/web/sql/260811_pre_deploy.sqlapps/web/sql/260811_post_deploy.sql다음 데이터는 의도적으로 유지했습니다.
account.social_infoclub.descriptionclub.articleclub.article_uploaded_atold_club_manager삭제 예정인 DB에 대한 참고: https://app.notion.com/p/DB-ER-32c70e37e9c7807f8b1bfdda66a0d879?source=copy_link#3b970e37e9c780ad9b72e6e66390196d
변경 이유
v2 이전 스키마와 API에서 사용하던 필드·테이블·엔드포인트가 현재 코드에 남아 있어 유지보수 비용과 DB 스키마 복잡도가 증가하고 있었습니다.
현재 앱과 서버에서 사용하지 않는 레거시 의존성을 제거하고, 사용자 학과 정보와 운영진 권한 신청 흐름을 신규 구조로 일원화하기 위해 변경했습니다.
테스트한 내용
pnpm verifybiome check284개 파일 통과git diff --check통과/api/v1server/service/v1entities/v1DeviceEntity,updateDeviceclub_manager_register_request배포 시 주의사항
260811_pre_deploy.sql실행develop에 머지하고 dev 서버 배포260811_post_deploy.sql실행post-deploy.sql실행 후에는 삭제된 컬럼을 참조하는 이전 서버 버전으로 단순 롤백할 수 없으므로, 모든 인스턴스 교체와 정상 동작을 확인한 뒤 실행해야 합니다.체크