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.

Sign up for GitHub

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

Jump to bottom

[개선] 단체 그룹 메시지 보내기 기능 리팩토링 #95

Merged
merged 5 commits into from
Jul 6, 2024
Merged

[개선] 단체 그룹 메시지 보내기 기능 리팩토링 #95

merged 5 commits into from
Jul 6, 2024

Conversation

jyj1289
Copy link
Member

@jyj1289 jyj1289 commented Jul 6, 2024

🎫 관련 이슈

close #94


📄 개요

단체 메시지 보내기 코드를 리팩토링했습니다.


🔨 작업 내용

  • formRepository에 메서드를 추가하였습니다.
  • changedToRegular를 이용하여 메시지를 보내게 수정하였습니다.
  • 불필요한 db 변경을 제거했습니다.

🏁 확인 사항

  • 테스트를 완료했나요?
  • API 문서를 작성했나요?
  • 코드 컨벤션을 준수했나요?
  • 불필요한 로그, 주석, import 등을 삭제했나요?

🙋🏻 덧붙일 말

@jyj1289
refactor(#94) : 불필요한 코드 삭제
30ce078 
- db 상에 changed_to_regular 컬럼이 있기 때문에 type을 굳이 변경할 필요가 없다고 생각되어 삭제하였습니다.
@jyj1289
feat(#94) : formRepository method 추가
3e0a18c 
- 기존의 stream을 통해서 마이스터인재전형에서 일반전형으로 전환된 원서를 찾았으나, 이 방법이 비효율적이라고 생각되어 changedToRegular 컬럼을 이용해서 formList를 반환하는 메서드를 새로 만들었습니다.
@jyj1289 jyj1289 requested a review from a team as a code owner July 6, 2024 08:00
@jyj1289 jyj1289 linked an issue Jul 6, 2024 that may be closed by this pull request
[개선] 단체 그룹 메시지 보내기 기능 리팩토링 #94
Closed
@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 6, 2024

이렇게 리뷰해주세요.

🖐️ 규칙

  1. 왜 개선이 필요한지 충분히 설명해주세요.
  2. 답을 알려주기보다는 스스로 고민하고 개선 방법을 선택할 수 있게 해주세요.
  3. 코드를 깔끔하게 유지하고, 일관되게 구현하도록 안내해 주세요.
  4. 리뷰 과정이 숙제검사가 아닌 학습과정으로 느낄 수 있게 리뷰해 주세요.
  5. 리뷰를 위한 리뷰를 하지 마세요. 피드백 할 게 없으면 칭찬해 주세요.

🥬 우선순위

리뷰 중요도에 따라 다음과 같은 prefix를 붙여주세요.

예시 - P5) 쓸모없는 주석 삭제해주세요.

  • P1: 꼭 반영해주세요 (Request changes)
  • P2: 적극적으로 고려해주세요 (Request changes)
  • P3: 웬만하면 반영해 주세요 (Comment)
  • P4: 반영해도 좋고 넘어가도 좋습니다 (Approve)
  • P5: 그냥 사소한 의견입니다 (Approve)

@cabbage16 cabbage16 self-requested a review July 6, 2024 08:10
@cabbage16 cabbage16 assigned jyj1289 and unassigned cabbage16 Jul 6, 2024
@jyj1289
fix(#94) : form type 컬럼을 변경하도록 수정
9f196f0 
- 분석을 하기 위해서는 form type이 Regular로 바뀌는 것이 맞다고 판단됭더 다시 변경하였습니다.
@jyj1289
refactor(#94) : method 이름 변경
2396009 
- findNotExistsMeisterTalentFirstRoundForm이 이제 전환된 원서를 포함하지 않게 변경되어 이름을 변경하였습니다.
@jyj1289
fix(#94) : 메서드 이름변경
e56c936 
- 메서드 이름을 변경한 것은 테스트코드에도 적용했어요.
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Jul 6, 2024

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@cabbage16 cabbage16 merged commit a0be604 into develop Jul 6, 2024
2 checks passed
@jyj1289 jyj1289 deleted the refactor/#94 branch July 10, 2024 02:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cabbage16 cabbage16 Awaiting requested review from cabbage16

Assignees

@jyj1289 jyj1289

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[개선] 단체 그룹 메시지 보내기 기능 리팩토링
2 participants
@jyj1289 @cabbage16