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

[레리]Chapter 8. API 응답 통일 & 에러 핸들러 #37

Merged
merged 2 commits into from
Jun 4, 2024

Conversation

wnd01jun
Copy link

@wnd01jun wnd01jun commented Jun 3, 2024

#️⃣ 연관된 이슈

#32


✅ 워크북 체크리스트

  • 모든 핵심 키워드 정리를 마쳤나요?
  • 핵심 키워드에 대해 완벽히 이해하셨나요?
  • 이론 학습 이후 직접 실습을 해보는 시간을 가졌나요?
  • 미션을 수행하셨나요?
  • 미션을 기록하셨나요?

✅ 컨벤션 체크리스트

  • 디렉토리 구조 컨벤션을 잘 지켰나요?
  • pr 제목을 컨벤션에 맞게 작성하였나요?
  • pr에 해당되는 이슈를 연결하였나요?
  • 적절한 라벨을 설정하였나요?
  • 스터디원들에게 code review를 요청하기 위해 reviewer를 등록하였나요?
  • 닉네임/main 브랜치의 최신 상태를 반영하고 있는지 확인했나요?

📌 주안점

@wnd01jun wnd01jun added the ✨ Chapter08 챕터 8 label Jun 3, 2024
@wnd01jun wnd01jun requested review from kjmq1234 and sseongeun June 3, 2024 13:35
@wnd01jun wnd01jun self-assigned this Jun 3, 2024
Copy link
Contributor

@kjmq1234 kjmq1234 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link

@sseongeun sseongeun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kjmq1234 kjmq1234 merged commit 26ad50b into reli/main Jun 4, 2024
@kjmq1234 kjmq1234 deleted the reli/#8 branch June 4, 2024 03:14
@chaeyeon0130
Copy link
Member

레리님 최고!! 8주차 너무 수고하셨습니다!😍

reli/#8 브랜치 -> 여기서 #8은 이슈/PR 번호를 의미해요! 그래서 8번으로 생성된 pr, [reli] chapter 2. AWS #8 해당 pr에 연결됩니다🫠

8주차 이슈는 #32이니 reli/#32 요렇게 브랜치를 생성하는 것이 맞습니다!!

다음 9주차도 파이팅!!!🔥🔥

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants