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

[SL-15] application 수정 사항 수정 #50

Merged
merged 17 commits into from
Jan 2, 2024

Conversation

jinyoungpark00
Copy link
Collaborator

@jinyoungpark00 jinyoungpark00 commented Jan 2, 2024

티켓

[SL-15]

변경사항

  • 지원서 1페이지 isAgreed null값이어도 처리 가능하도록 수정
  • GET application과 GET application/{applicationId} 통합
  • generation 전역 변수로 설정
  • partQuestion 개수 4개로 증가
  • 기존 DTO의 partId를 part로 수정 후 string으로 받게 변경
  • pathToKnow에 etc_detail 컬럼 추가 (ETC 선택 시 추가 입력)
  • GET application 시 body 값만 리턴되도록 형식 수정

부가설명

  • isAgreed 값이 null이라면 false로 저장하도록 했습니다.
  • GET /application을 삭제하고 GET /application/{applicationId}에서 applicationId의 값이 0일 경우 이전의 GET /application 역할을 하도록 수정했습니다.
  • generation을 전역 변수로 설정해서 기존의 쿼리 파라미터로 받지 않고 내부적인 값으로 사용하도록 수정했습니다.

고려사항

  • properties 파일에 변동이 생겨서 머지 후 슬랙에 변동 사항 올려놓겠습니다.

스크린샷

@jinyoungpark00 jinyoungpark00 added the Chore minor modifications label Jan 2, 2024
@jinyoungpark00 jinyoungpark00 self-assigned this Jan 2, 2024
@bykbyk0401
Copy link
Collaborator

확인했습니다.
수고하셨습니다~

@jinyoungpark00 jinyoungpark00 merged commit 525a2b1 into develop Jan 2, 2024
1 check passed
@unanchoi unanchoi deleted the chore/SL-15/application branch January 2, 2024 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Chore minor modifications
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants