Skip to content

[week06]발베니 6주차 미션1#125

Open
Seongmin0218 wants to merge 1 commit into
mainfrom
week06_Balvenie_mission1
Open

[week06]발베니 6주차 미션1#125
Seongmin0218 wants to merge 1 commit into
mainfrom
week06_Balvenie_mission1

Conversation

@Seongmin0218
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

@Eunjin03 Eunjin03 left a comment

Choose a reason for hiding this comment

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

안녕하세요! 6주차도 고생많으셨습니다.

늘 코드를 엄청 깔끔하게 작성하시는 것 같습니다! 디렉토리 구조화도 잘 되어있구요 ㅎㅎ
lp detail page는 만드셨는데 실행 영상에 함께 첨부가 안되어있더라구요 ㅠㅠ 다음엔 꼭 같이 넣어서 보여주세요!! 사실 제가 보고싶어서요..ㅎㅎ

7주차도 화이팅입니다!


{isLoggedIn ? (
<>
{/* 마이페이지 */}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

미션에는 해당 부분이 "~님 환영합니다" 식으로 바뀌었더라고요! 반영 부탁드립니당

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

전반적으로 쿼리 코드 넘 잘짜셨네요! 파일 분리도 잘하셨습니다

</div>

<div className="grid grid-cols-2 gap-4 sm:grid-cols-[repeat(auto-fill,minmax(180px,1fr))] md:grid-cols-[repeat(auto-fill,minmax(200px,1fr))]">
{lpList.map((lp: LpItem) => {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lp component를 분리하여 사용하는 것도 좋은 방법일 것 같아요!

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.

2 participants