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 CHORE] application 수정 #37

Merged
merged 7 commits into from
Dec 29, 2023
Merged

Conversation

jinyoungpark00
Copy link
Collaborator

티켓

[SL-15]

변경사항

  • 배포 환경에서 h2 관련 에러가 나서 주석처리
  • 메인 페이지 controller 추가 및 인가 제외
  • 본인의 application만 접근 가능하도록 에러 처리
  • page1 PUT을 PUT과 POST로 분리
  • 코드 메서드 분리

부가설명

  • 본인의 application 외의 applicationId로 접근할 시 에러를 띄우도록 수정했습니다.
  • 기존의 방식은 PUT을 활용해서 해당 applicationId가 없는 경우 application의 컬럼을 만드는 방식이었습니다. 하지만 이 방식을 사용하면 지원서를 생성할 때 applicationId를 결정해줘야 한다는 문제가 있습니다. 그래서 GET application/ 으로 접근할 때는 POST를 사용해서 application 컬럼을 만든 후 id를 반환하도록 했고, GET application/{applicationId}로 접근한 경우에는 id값이 있으므로 PUT을 사용해서 업데이트 하도록 수정했습니다.

고려사항

  • 더 좋은 방법이 있거나 수정사항 있으면 알려주세요!

스크린샷

@jinyoungpark00 jinyoungpark00 added the Chore minor modifications label Dec 28, 2023
@jinyoungpark00 jinyoungpark00 self-assigned this Dec 28, 2023
@eunsu02
Copy link
Member

eunsu02 commented Dec 29, 2023

확인했습니다! 수고 하셨습니다~
수정 된 방법이 더 깔끔한것 같네요!

@jinyoungpark00 jinyoungpark00 merged commit 36a2b73 into develop Dec 29, 2023
1 check passed
@jinyoungpark00 jinyoungpark00 deleted the chore/SL-15/application branch December 29, 2023 12:35
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