커밋 유실기록 복구 #85

jyj1289 · 2024-06-11T14:07:15Z

🎫 관련 이슈

close #

📄 개요

🔨 작업 내용

🏁 확인 사항

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

🙋🏻 덧붙일 말

- 메시지를 그룹별로 보낼 수 있는 메세지 컨트롤러를 만들었습니다.

- 메시지를 그룹별로 보낼 수 있는 메세시를 보낼 수 있는 서비스를 만들었습니다.

- cors에 마루 버셀 url을 추가했습니다.

- MessageControllerTest를 만들었어요.

- MessageController에 대한 아스키 독스를 만들었어요

- 횟수 검증을 할 필요가 없어서 verify() 호출 코드를 제거했어요

- 불필요한 주석을 제거했어요

- 불필요한 중복 코드를 제거했어요

- 코드를 정렬했어요

- SendMessageUseCase를 만들었어요

- execute에서 phoneNumberList를 생성하던 것을 UseCase로 이전했어요. - 기존 for문을 stream으로 변환했어요.

- SendMessageService를 사용하던 코드들을 SendMessageUseCase를 사용하도록 변환했어요.

- stream 컨벤션을 사용했어요.

- 원서 상태별로 메시지 보내기와 원서 타입별로 메시지 보내기 기능을 분리하기 위해 이름과 엔드포인트를 변경했어요.

- BDD관점으로 보기 위해서 BDDMockito메서드로 변경했어요. - UseCaseTest에서 status별로 테스트하기 위해서 컨트롤러 테스트를 수정했어요.

- SendMessageUseCaseTest를 만들었어요.

Feat/#75

- SendMessageService에서 formRepository를 사용하지 않아서 제거했어요.

- 마이스터전형과, 마이스터전형을 제외한 모든 전형을 조회하기 위해서 findMeisterTalentFirstRoundForm()과 findNotExistsMeisterTalentFirstRoundForm() 메서드를 추가했어요

- 마이스터전형과, 마이스터전형을 제외한 모든 전형을 조회하기 위해서 findMeisterTalentFirstRoundForm()과 findNotExistsMeisterTalentFirstRoundForm() 메서드를 구현했어요.

- 전형별로 메시지를 보내기 위해서 컨트롤러에 메서드를 추가했어요.

- SendMessageByStatusRequest로 이름을 변경했어요

- 전형별 메시지 보내기 메서드를 만들었어요. - 비즈니스 로직을 private 메서드로 바꾸었어요. - 겹치는 로직을 하나의 메서드로 개선했어요

…feat/#81

- SendMessageByTypeRequest DTO를 추가했어요.

- SendMessageRequest를 SendMessageByStatusRequest로 변경했어요

- 불필요한 import를 제거했어요.

- 마이스터 전형에서 일반전형으로 바뀐 원서는 getType()을 사용했을 때만 REGULAR로 나오고 실제 디비에는 MEISTER_TALENT로 들어가서 그룹별 메시지를 보낼 때 번거로움이 있어서 원서 전형 자체를 REGULAR로 바꾸기로 결정했어요.

- 요청할 때 인식이 되지 않아서 boolean 타입에서 Boolean 타입으로 바꾸었어요

- /message/type 테스트를 추가했어요.

- 테스트를 위한 println코드를 삭제했어요

- 전형별 메시지 보내기 테스트 코드를 작성했어요.

github-actions · 2024-06-11T14:07:30Z

이렇게 리뷰해주세요.

🖐️ 규칙

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

🥬 우선순위

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

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

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

sonarcloud · 2024-06-11T14:07:47Z

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

jyj1289 and others added 30 commits May 1, 2024 19:55

feat : MessageController

86dac3b

- 메시지를 그룹별로 보낼 수 있는 메세지 컨트롤러를 만들었습니다.

feat(#75):send message by group service

9d04ef8

- 메시지를 그룹별로 보낼 수 있는 메세시를 보낼 수 있는 서비스를 만들었습니다.

docs:add cors url

9b737c7

- cors에 마루 버셀 url을 추가했습니다.

feat(#75):MessageControllerTest

71538fa

- MessageControllerTest를 만들었어요.

docs(#75):MessageController asciidocs

92012f5

- MessageController에 대한 아스키 독스를 만들었어요

fix(#75) : remove verify()

f738bac

- 횟수 검증을 할 필요가 없어서 verify() 호출 코드를 제거했어요

refactor(#75) : 주석 제거

2e785fe

- 불필요한 주석을 제거했어요

refactor(#75) : 중복 코드 제거

8bc31ad

- 불필요한 중복 코드를 제거했어요

refactor(#75) : 자동 정렬

64a20e1

- 코드를 정렬했어요

feat(#75) : SendMessageUseCase

30cfb0e

- SendMessageUseCase를 만들었어요

refact(#75) : MessageUseCase 사용

b67cec0

- execute에서 phoneNumberList를 생성하던 것을 UseCase로 이전했어요. - 기존 for문을 stream으로 변환했어요.

refact(#75) : MessageUseCase 사용

de4965e

- SendMessageService를 사용하던 코드들을 SendMessageUseCase를 사용하도록 변환했어요.

refactor(#75) : stream 컨벤션 사용

5ea895a

- stream 컨벤션을 사용했어요.

refactor(#75) : sendMessage 메서드 이름 변경 및 엔드포인트 변경

5ed2f44

- 원서 상태별로 메시지 보내기와 원서 타입별로 메시지 보내기 기능을 분리하기 위해 이름과 엔드포인트를 변경했어요.

test(#75) : 엔드포인트 변경 및 BDDMockito 메서드로 변경

411cad3

- BDD관점으로 보기 위해서 BDDMockito메서드로 변경했어요. - UseCaseTest에서 status별로 테스트하기 위해서 컨트롤러 테스트를 수정했어요.

test(#75) : status별 단위 테스트 작성

fddf9a3

- SendMessageUseCaseTest를 만들었어요.

Merge pull request #80 from Bamdoliro/feat/#75

d8b3e46

Feat/#75

feat(#81) : 불필요한 코드 개선

e57d2a9

- SendMessageService에서 formRepository를 사용하지 않아서 제거했어요.

feat(#81) : 메서드 추가

53e18ec

- 마이스터전형과, 마이스터전형을 제외한 모든 전형을 조회하기 위해서 findMeisterTalentFirstRoundForm()과 findNotExistsMeisterTalentFirstRoundForm() 메서드를 추가했어요

feat(#81) : formRepository 메서드 구현

626a4ee

- 마이스터전형과, 마이스터전형을 제외한 모든 전형을 조회하기 위해서 findMeisterTalentFirstRoundForm()과 findNotExistsMeisterTalentFirstRoundForm() 메서드를 구현했어요.

feat(#81) : 전형별 메세지 메서드 추가

1eb26e2

- 전형별로 메시지를 보내기 위해서 컨트롤러에 메서드를 추가했어요.

refactor(#81) : DTO 이름 변경

d6e12fd

- SendMessageByStatusRequest로 이름을 변경했어요

feat(#81) : execute 메서드 추가 및 개선

13a9a70

- 전형별 메시지 보내기 메서드를 만들었어요. - 비즈니스 로직을 private 메서드로 바꾸었어요. - 겹치는 로직을 하나의 메서드로 개선했어요

docs : message.adoc 변경

7126186

Merge branch 'develop' of https://github.com/Bamdoliro/marubase into …

5beb8ba

…feat/#81

feat(#81) : SendMessageByTypeRequest 추가

68b8f9a

- SendMessageByTypeRequest DTO를 추가했어요.

fix(#81) : SendMessageRequest 이름 변경

135cf36

- SendMessageRequest를 SendMessageByStatusRequest로 변경했어요

refactor(#81) : import 제거

79cec34

- 불필요한 import를 제거했어요.

fix(#81) : boolean 타입 변경

48f997e

- 요청할 때 인식이 되지 않아서 boolean 타입에서 Boolean 타입으로 바꾸었어요

jyj1289 added 4 commits May 31, 2024 17:13

docs(#81) : message.adoc

1fe3954

- 요청할 때 인식이 되지 않아서 boolean 타입에서 Boolean 타입으로 바꾸었어요

feat(#81) : messageControllerTest 추가

7d8f19e

- /message/type 테스트를 추가했어요.

fix(#81) : 테스트 코드 삭제

dc71040

- 테스트를 위한 println코드를 삭제했어요

feat(#81) : 전형별 메시지 보내기 테스트 코드 작성

ce3304e

- 전형별 메시지 보내기 테스트 코드를 작성했어요.

jyj1289 requested a review from a team as a code owner June 11, 2024 14:07

jyj1289 merged commit 784f373 into develop Jun 11, 2024
5 checks passed

jyj1289 deleted the feat/#81 branch June 11, 2024 14:09

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

커밋 유실기록 복구 #85

커밋 유실기록 복구 #85

jyj1289 commented Jun 11, 2024

github-actions bot commented Jun 11, 2024

sonarcloud bot commented Jun 11, 2024

커밋 유실기록 복구 #85

커밋 유실기록 복구 #85

Conversation

jyj1289 commented Jun 11, 2024

🎫 관련 이슈

📄 개요

🔨 작업 내용

🏁 확인 사항

🙋🏻 덧붙일 말

github-actions bot commented Jun 11, 2024

이렇게 리뷰해주세요.

🖐️ 규칙

🥬 우선순위

sonarcloud bot commented Jun 11, 2024

Quality Gate passed