Skip to content

[1주차/제이] 워크북 제출합니다 - #20

Merged
yeongjun25 merged 4 commits into
UMC-Inha:j/mainfrom
yeongjun25:main
Mar 24, 2026
Merged

yeongjun25 merged 4 commits into
UMC-Inha:j/mainfrom
yeongjun25:main

Conversation

@yeongjun25

@yeongjun25 yeongjun25 commented Mar 18, 2026 •

Copy link
Copy Markdown

✅ 워크북 체크리스트

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

✅ 컨벤션 체크리스트

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

📌 주안점

@yeongjun25
yeongjun25 requested a review from YoungJJun March 18, 2026 14:57
@yeongjun25 yeongjun25 self-assigned this Mar 18, 2026
@yeongjun25 yeongjun25 linked an issue Mar 18, 2026 that may be closed by this pull request

@YoungJJun YoungJJun left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

1주차 피드백

머지전에 피어리뷰 추가해주세요~


1번 쿼리 좋아요~

2번

  • 가게 이름이 빠진것 같아요.
  • member_mission_id는 따로 가져오신 이유가 있으실까요? 커서 페이징 사용시 어디까지 가져왔는지 확인하기 위해 필요할수도 있는데 여기서는 오프셋 페이징인거 같아서요!
  • store_id를 같이 가져오신건 리뷰를 남기기 위해서가 맞을까요?! 맞으면 너무 좋은생각 같습니다.

3번 좋습니다.

4번

  • 위쪽 페이지에 대해서는 mod 10 해서 넘겨줘도 되고 프론트에서 처리하도록 해도 될 것 같아요. 좋습니다!

  • 아래부분에 대해 도전 가능한 미션을 보여주는거라 이미 완료되었거나 진행중인 미션은 제외해야 할 것 같아요.

  • JOIN 에서 regions AS j 라고 선언하셨는데 바로 아래 where에서 r. 으로 시작해요. 이 경우 r이라는 테이블은 없기 때문에 실행 안될거에요. r이나 j로 통일하시면 될 것 같아요.


제이 1주차 수고하셨습니다. 2주차도 화이팅입니다!

@yeongjun25
yeongjun25 merged commit e185cde into UMC-Inha:j/main Mar 24, 2026
@yeongjun25

Copy link
Copy Markdown
Author

자세한 피드백 감사합니다!

  1. 쿼리 짤 때 member_mission_id가 필요하다고 해서 적었다가 지우는 걸 깜빡했습니다. (실제로 사용하지 않는 데이터입니다!)
  2. 리뷰 남기기를 위해서 store_id를 가져온 거 맞습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Chapter01_DB와 친해지기 - SQL문

2 participants