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

[JT-37] 웹툰 정보 조회 기능 구현 #12

Merged
merged 4 commits into from
Sep 7, 2023
Merged

Conversation

Shin-Jae-Yoon
Copy link
Collaborator

📌 개발 내용

  • spring-cloud-aws는 s3 설정을 yml 기반으로 자동으로 해주어서 s3 config를 삭제했습니다.
  • s3에 올라간 이미지가 보여지는 것이 아니라 다운로드가 바로 되어서 contentType을 수정했습니다.
  • 로그인 한 회원만 웹툰을 생성할 수 있도록 수정하였습니다.
  • 웹툰 정보 조회 기능을 구현했습니다.

📑 PR 포인트

  • 우리 웹툰팀은 toDto가 아닌 from을 지지하기로 했습니다. ^^

👥 협업을 위한 코드리뷰

  1. 세상에 바보같은 질문은 없다.
  2. 실수를 예방하는 팀이 아니라, 실수를 잘 다루는 팀이 되자.
  3. 분업이 아닌 협업을 하자. 우린 모두 같은 문제를 이겨내기 위해 모였다.
  4. 영원한 것은 없으니 대화를 하자.
  5. 서로 존중하자.
  6. 많이 부딪치고 깨닫고 배우자.
  7. 부드럽게 설득하고 열린 마음으로 설득당해보자.

✅ 리마인더

  • 본인의 로컬에서 정상 동작하는지 확인해주세요.
  • 최신 브랜치를 Pull 받고 PR을 요청했는지 확인해주세요.
  • API가 추가되었을 경우 테스트를 하고 PR을 올려주세요.
  • Conflict가 났을 때, UI상에서 해결하지 말고, 본인 local에서 해결해주세요.
  • Commit 메시지 제대로 작성해주세요.

⚙️ 코드리뷰 룰

  • R(Request Change): 해당 블럭은 꼭 변경해주셨으면 좋겠습니다.
  • C(Comment): 웬만하면 고려해주시면 좋겠습니다.
  • Q(Question) : 해당 라인이 궁금합니다.
  • A(Approve): 반영해도 좋고 넘어가도 좋습니다. 혹은 사소한 의견입니다.

@Shin-Jae-Yoon Shin-Jae-Yoon merged commit 04b0f34 into develop Sep 7, 2023
1 check passed
@kmebin kmebin deleted the feature/JT-37 branch September 14, 2023 07:54
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.

3 participants