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

Commits on Jul 6, 2024

  1. refactor(#94) : 불필요한 코드 삭제 

    - db 상에 changed_to_regular 컬럼이 있기 때문에 type을 굳이 변경할 필요가 없다고 생각되어 삭제하였습니다.
    @jyj1289
    jyj1289 committed Jul 6, 2024
    Configuration menu
    Copy the full SHA
    30ce078 View commit details
    Browse the repository at this point in the history

  2. feat(#94) : formRepository method 추가 

    - 기존의 stream을 통해서 마이스터인재전형에서 일반전형으로 전환된 원서를 찾았으나, 이 방법이 비효율적이라고 생각되어 changedToRegular 컬럼을 이용해서 formList를 반환하는 메서드를 새로 만들었습니다.
    @jyj1289
    jyj1289 committed Jul 6, 2024
    Configuration menu
    Copy the full SHA
    3e0a18c View commit details
    Browse the repository at this point in the history

  3. fix(#94) : form type 컬럼을 변경하도록 수정 

    - 분석을 하기 위해서는 form type이 Regular로 바뀌는 것이 맞다고 판단됭더 다시 변경하였습니다.
    @jyj1289
    jyj1289 committed Jul 6, 2024
    Configuration menu
    Copy the full SHA
    9f196f0 View commit details
    Browse the repository at this point in the history

  4. refactor(#94) : method 이름 변경 

    - findNotExistsMeisterTalentFirstRoundForm이 이제 전환된 원서를 포함하지 않게 변경되어 이름을 변경하였습니다.
    @jyj1289
    jyj1289 committed Jul 6, 2024
    Configuration menu
    Copy the full SHA
    2396009 View commit details
    Browse the repository at this point in the history

  5. fix(#94) : 메서드 이름변경 

    - 메서드 이름을 변경한 것은 테스트코드에도 적용했어요.
    @jyj1289
    jyj1289 committed Jul 6, 2024
    Configuration menu
    Copy the full SHA
    e56c936 View commit details
    Browse the repository at this point in the history