Skip to content

[Refactor] 축제 관련 api 변경에 따른 마이그레이션#54

Merged
etama123 merged 9 commits intodevelopfrom
refactor/53
Feb 16, 2026
Merged

[Refactor] 축제 관련 api 변경에 따른 마이그레이션#54
etama123 merged 9 commits intodevelopfrom
refactor/53

Conversation

@etama123
Copy link
Contributor

#️⃣ 이슈 번호

#53


🛠️ 작업 내용

  • 서버 마이그레이션에 따른 api 호출 부분을 변경했습니다.

🙇🏻 중점 리뷰 요청

  • 안드 프로젝트도 업데이트된 내용으로 local.properties를 변경해주시기 바랍니다.
  • kmp pr과 달리, 해당 레포지터리에서는 기존 도메인 구조를 수정하지 않았습니다.

- `festivals/universities`에서 `/organizations/festivals/search`로 API 엔드포인트를 수정했습니다.
- API 변경에 따라 `universityName` 쿼리 파라미터를 `keyword`로 변경했습니다.
API 명세 변경에 따라, `FestivalResponse`, `UniversityResponse`, `RegisteredFestivalNotificationResponse` 모델의 `universityName` 필드를 `organizationName`으로 수정했습니다.
@etama123 etama123 self-assigned this Feb 14, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 14, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch refactor/53

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.

Copy link
Contributor

@parkjiminnnn parkjiminnnn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

허허 CI가 터졌네요잉

- CI 워크플로우에서 `IMAGE_BASE_URL`과 `IMAGE_BASE_URL_DEV`를 `local.properties` 파일에 추가하도록 수정했습니다.
`build.gradle.kts` 파일에서 개발용(`_DEV`) 속성을 가져올 때, 속성이 누락된 경우 출력되는 에러 메시지를 보다 명확하게 수정했습니다
CI 워크플로우에서 `local.properties` 파일을 생성할 때, `BASE_URL`과 `BASE_URL_DEV` 환경 변수 할당 방식에 있던 불필요한 따옴표를 제거하여 다른 변수들과 형식을 통일했습니다.
CI 워크플로우에서 `local.properties` 파일을 생성하는 스크립트의 오류를 수정했습니다.

- `BASE_URL`과 `IMAGE_BASE_URL`을 파일에 쓸 때, 기존 내용을 덮어쓰는(`>`) 대신 추가하도록(`>>`) 변경했습니다.
API 명세 변경에 따라, `FestivalNotificationRepositoryTest`에서 사용하는 `RegisteredFestivalNotificationResponse`의 테스트 데이터를 수정했습니다.

- `universityName` 필드를 `organizationName`으로 변경했습니다.
- `festivalName` 필드에 테스트 값을 추가했습니다.
@etama123 etama123 merged commit fad4f84 into develop Feb 16, 2026
3 checks passed
@etama123 etama123 deleted the refactor/53 branch February 16, 2026 11:33
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.

3 participants