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

refact: 백엔드 리팩토링 적용 #30

Merged
merged 2 commits into from
Oct 7, 2023
Merged

refact: 백엔드 리팩토링 적용 #30

merged 2 commits into from
Oct 7, 2023

Conversation

gyeong3un2
Copy link
Member

🛠️ 변경사항

  • Post dto name 필드 -> title로 변경
  • Post 조회시 응답 형태 변경
  • 게시글 생성과 룩북 생성 시 request에서 userId 삭제하고, header에서 Authorization: token으로 주는 거로 변경
  • 그리고 Authorization이 필요한 API에다가 header에서 Authorization: token 추가
  • 날씨별 게시글 조회시 하늘상태 + 강수형태 필드 이름(Enum) 변경
  • 유저 아이디로 룩북 불러오는 API 주소와 룩북 아이디로 룩북 리스트 불러오는 API 주소 수정


☝️ 유의사항



❗체크리스트

  • 하나의 메소드는 최소의 기능만 하도록 설정했나요?
  • 수정 가능하도록 유연하게 작성했나요?
  • 필요 없는 import문이나 setter 등을 삭제했나요?
  • 기존의 코드에 영향이 없는 것을 확인하였나요?

@gyeong3un2 gyeong3un2 linked an issue Oct 7, 2023 that may be closed by this pull request
@gyeong3un2 gyeong3un2 merged commit 3a6183a into develop Oct 7, 2023
1 check passed
@gyeong3un2 gyeong3un2 deleted the refact/i29 branch October 7, 2023 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[REFACTOR] 백엔드 변경사항 적용
1 participant