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

Umi/4week #24

Merged
merged 3 commits into from
May 9, 2024
Merged

Umi/4week #24

merged 3 commits into from
May 9, 2024

Conversation

jejejekim
Copy link

@jejejekim jejejekim commented May 7, 2024

개요⭐️

#23 미션 완료 했습니다!

작업사항⭐️

  • 로그인/로그아웃 onClick
스크린샷 2024-05-07 오후 1 04 05
  • 상세페이지 MovieDetailPage
스크린샷 2024-05-07 오후 1 03 50
  • 에러페이지 NotFoundPage(ErrorPage)
스크린샷 2024-05-07 오후 1 04 45

확인⭐️

  • 워크북 키워드 정리 완료
  • 실습 및 미션 완료

@jejejekim jejejekim self-assigned this May 9, 2024
Copy link
Contributor

@yeon-dong yeon-dong 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

@ssolfa ssolfa 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
Contributor

@sihyun2988 sihyun2988 left a comment

Choose a reason for hiding this comment

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

LGTM~~!!

@jejejekim jejejekim merged commit d578980 into umi/main May 9, 2024
@jejejekim jejejekim deleted the umi/4week branch May 12, 2024 13:12
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