Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BE] refactor: 리뷰 그룹에 템플릿 정보를 하드코딩 하지 않도록 변경 #812

Merged
merged 2 commits into from
Oct 14, 2024

Conversation

Kimprodp
Copy link
Contributor


🚀 어떤 기능을 구현했나요 ?

  • 현재 리뷰 그룹에 templateId가 1로 하드코딩 되어있습니다. 추후 확장성 고려 및 테스트에서 템플릿 id를 직접 넣어서 다양한 케이스를 테스트할 수 있도록 생성자를 통해서 등록하는 것으로 변경합니다.
  • 우선은 템플릿 확장에 대한 논의가 없기 때문에 reviewGorupService에서 리뷰 그룹 등록 시, 1번 템플릿을 디폴트로 리뷰 그룹을 등록하는 것으로 합니다.

🔥 어떻게 해결했나요 ?

  • reviewGroup 생성자에 templateId 추가

📝 어떤 부분에 집중해서 리뷰해야 할까요?

📚 참고 자료, 할 말

Copy link

github-actions bot commented Oct 10, 2024

Test Results

146 tests  ±0   146 ✅ ±0   4s ⏱️ -1s
 55 suites ±0     0 💤 ±0 
 55 files   ±0     0 ❌ ±0 

Results for commit 1870f28. ± Comparison against base commit b6d03d1.

♻️ This comment has been updated with latest results.

@Kimprodp Kimprodp force-pushed the be/refactor/809-reviewGroup-templateId branch from 3b6faa6 to 1870f28 Compare October 11, 2024 08:46
Copy link
Contributor

@nayonsoso nayonsoso left a comment

Choose a reason for hiding this comment

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

어푸루부입니다~~

@Kimprodp Kimprodp merged commit d93a1a7 into develop Oct 14, 2024
5 checks passed
@donghoony donghoony deleted the be/refactor/809-reviewGroup-templateId branch October 14, 2024 12:26
skylar1220 pushed a commit that referenced this pull request Oct 15, 2024
* refactor: 리뷰 그룹 생성 시, 템플릿 id를 설정하도록 변경

* test: 변경 사항 테스트 반영
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[BE] 리뷰 그룹에 템플릿 하드 코딩을 제거한다.
4 participants