Skip to content

prod 배포#106

Closed
mjy926 wants to merge 36 commits into
mainfrom
develop
Closed

prod 배포#106
mjy926 wants to merge 36 commits into
mainfrom
develop

Conversation

@mjy926

@mjy926 mjy926 commented Sep 9, 2025

Copy link
Copy Markdown
Contributor
  • 아키텍쳐 변경을 위해 prod에도 배포하겠습니다.

minkyu97 and others added 30 commits August 12, 2024 15:27
- SSE 기본 ping 이벤트 간격 3600초로 변경
- CodeSubmissionResult 가 INTERNAL_SERVER_ERROR 일 때 CodeSubmission 를
  Error로 판단
- member, project, history, recruting_info, review, seminar, sponsor,
timeline, faq 테이블에 대해 모델 정의를 작성했습니다.
- recruit와 recruiting_info 관계를 정의했습니다.
- pre-commit hook에 있는 pycln 버전에 문제가 있어 2.5.0 버전으로 변경했습니다.
- migration 파일을 추가했습니다.
- Project Image URL 테이블을 추가했습니다.
- timeline 테이블에서 content 필드를 삭제했습니다.
- pre_registration 테이블을 추가했습니다.
- project api와 recruiting api를 구현했습니다.

- 어드민 권한이 필요한 api의 경우에는 수작업으로 user table에서 is_admin 값이 true인 데이터를 만들어서
테스트 해보았습니다
      로컬 회원가입을 사용하기로 하였으니 이후에 이와 관련한 작업이 필요할 것 같습니다.
- 만들고 보니 recruiting_info를 생성하는 api가 없습니다. recruiting_info 생성에 대한 api가
필요할 것 같습니다
- 지금 현재 리크루트 업데이트 api에서 resume_questions이나 problems의 개별 항목을 업데이트 하지는
못하고, 기존 질문들을 모두 제거하고 새로 추가하는 식으로 구현되어 있습니다
    
 - ProjectMember 테이블을 추가했습니다.
 - Project 테이블에서 project_type, is_active 필드를 추가했습니다.
 - ProjectUrl 테이블에서 title 필드를 추가했습니다.
* seminar api를 구현했습니다.
* seminar 모델에서 변경할 것들이 있어 변경 후 마이그레이션 파일 추가했습니다.
- enum class의 구조를 변경하였습니다
- uri 중복으로 인한 오류로 인해 uri 주소를 변경했습니다
- member api를 구현했습니다
    - get_members 함수의 경우 헤더로 받는 sso_id가 어드민인지 아닌지에 따라 반환하는 응답이 다릅니다

- erd 그림에는 포함되어 있지만 member model에 introduction이 추가되어 있지 않아서 introduction
필드를 추가했습니다.

커밋이 약간 꼬여서 member api 구현 커밋이 두번 있게 됐습니다...
- 리뷰 api를 구현했습니다
* History 관련 api를 구현했습니다.
* History table을 수정하고 마이그레이션 파일을 추가했습니다.
* project url type을 enum으로 수정 후 마이그레이션 파일을 추가했습니다.
* question과 관련된 api를 구현했습니다.
- timeline과 timeline category api를 구현했습니다
- 노션 api 요구사항에 맞게 Timeline 테이블에서 category 열을 없애고 category_id 외래키 열을
추가했습니다
- 서비스/스터디 목록 불러오는 api를 페이지네이션 가능하게 변경했습니다
- 여러 파일에 오타를 수정했습니다
- 모집 일정 불러오는 api에서 recruiting_type을 query parameter로 변경했습니다
* 리스트가 비어있을 때의 예외 처리를 삭제했습니다.
* pre registration api를 구현했습니다.
* pre_registration 테이블이 추가가 안되어있어 마이그레이션 파일을 추가했습니다.
- 프로젝트 이미지 업로드를 위한 api를 구현했습니다
- 프로젝트 이미지가 저장되는 버킷은 기존의 포트폴리오가 저장되는 버킷인 wacruit-protfolio-dev 또는
wacruit-protfolio-prod입니다.
    - 프로젝트 이미지의 key의 형식은 PROJECT/{project_id}/{file_name}입니다.
    - 프로젝트 이미지를 업로드할 때는 프로젝트가 먼저 생성되어 있어야 합니다
    - api의 세부적인 내용은 노션을 참조해주세요
- project_image 테이블에서 url 대신 s3 객체 키를 저장하도록 수정하였습니다.
- 어드민 페이지에 새로 추가된 api들이 보일 수 있도록 수정하였습니다
- HistoryResponse를 수정했습니다.
- Project introduction과 Review content의 길이가 짧은 것 같아 수정했습니다
* 라우터가 중복으로 등록되어 생기는 경고 메세지가 나오지 않도록 수정했습니다
* Recruiting type을 prod와 맞춰주었습니다.
* stdin, expected_output을 MediumText로 수정했습니다.

---------

Co-authored-by: deveroskp <imsmile2004@naver.com>
* testcase의 stdin, expected_output이 Textarea로 보이도록 수정했습니다.
* testcase를 삭제할 때 해당하는 code submission result도 같이 삭제되도록 수정했습니다.
* 타임라인 테스트 코드를 작성했습니다
- pre-registration pytest 코드를 작성했습니다
- sponsor 테스트 코드를 작성했습니다
- sponsor api 코드를 수정했습니다
    - amount 자료형을 int으로 하고,  sponsored_date의 자료형을 Date로 바꿨습니다
    - 제가 그 당시에 코드를 대충 짰나 봅니다 ㅠㅠ

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

- Refactor
  - Standardized sponsor amount to an integer value.
  - Sponsored date is now date-only (no time/timezone).
  - API endpoints now accept numeric sponsor IDs instead of strings.
- Tests
- Added comprehensive tests for sponsor creation, retrieval, listing,
and updates.
- Chores
  - Database schema updated to align column types with the new formats.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
mjy926 and others added 5 commits August 31, 2025 14:15
* 세미나 API에 대해 테스트 코드를 추가했습니다.

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

- Tests
- Added comprehensive tests for seminar management, covering creation,
retrieval (active, all, by ID), and updates to ensure correct behavior
and data integrity.
- Introduced reusable test fixtures to standardize test setup and
improve consistency.
- Expanded coverage across active and inactive scenarios to increase
reliability.
- Chores
- No user-facing changes; improvements focus on test robustness and
maintainability.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
- faq 테스트 코드를 작성했습니다
- faq api에서 QuestionNowFoundException를 QuestionNotFoundException로 바꿨습니다
- faq 생성 api에서 Service 레이어에 request를 넘겨주는 것으로 수정했습니다

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* Bug Fixes
* Ensured FAQ “question not found” responses consistently return 404
with a clear message.

* Refactor
* Streamlined FAQ creation flow to improve consistency and
maintainability.

* Tests
* Added comprehensive tests covering FAQ create, read, update, and
delete scenarios.
  * Updated sponsor tests to support and verify partial updates.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
* coderabbit yaml 파일을 추가했습니다.
- review를 불러오는 api에서 멤버의 기수를 불러오도록 수정하였습니다.
@mjy926 mjy926 requested a review from a team as a code owner September 9, 2025 05:32
@coderabbitai

coderabbitai Bot commented Sep 9, 2025

Copy link
Copy Markdown

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.

✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch develop

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.

* develop -> main 으로 갈 때 충돌이 일어나서 수정했습니다.

---------

Co-authored-by: MINKYU LEE <whizkyu@snu.ac.kr>
Co-authored-by: fivessun-ridi <fivessun@ridi.com>
@mjy926 mjy926 closed this Sep 11, 2025
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.

3 participants