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

[JR-879] 로그인 되어있지 않을시 나타나는 모달 구현 #202

Merged
merged 5 commits into from
Oct 18, 2023

Conversation

Leejha
Copy link
Member

@Leejha Leejha commented Oct 18, 2023

📑 작업리스트

  • 유저 권한이 없을시 로그인 페이지로 이동시키는 모달 적용
  • 마이페이지 바텀 네비게이션바의 의해 잘리지 않도록 수정

📘 리뷰노트

@vercel
Copy link

vercel bot commented Oct 18, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
jurumarble ❌ Failed (Inspect) Oct 18, 2023 4:46pm
web ❌ Failed (Inspect) Oct 18, 2023 4:46pm

Copy link
Member

@leejiho9898 leejiho9898 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다

@@ -59,7 +65,7 @@ const DrinkInfoContainer = () => {
</PreviousButton>
}
rightButton={
<PreviousButton onClick={() => router.back()}>
<PreviousButton onClick={() => router.push(Path.SEARCH_PAGE)}>
Copy link
Member

Choose a reason for hiding this comment

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

안드로이드 뒤로가기키 클릭과 privious button 동작이 달라질텐데 의도된 동작이 맞을까요?

Copy link
Member Author

Choose a reason for hiding this comment

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

아 그런데 저게 검색 아이콘이더라고요
이전 버튼은 왼쪽에 있습니다!

@Leejha Leejha force-pushed the JR-879-qa-process-non-login branch from 1d5d95e to 1940d89 Compare October 18, 2023 16:42
@Leejha Leejha merged commit 9843b1a into develop Oct 18, 2023
2 of 3 checks passed
@Leejha Leejha deleted the JR-879-qa-process-non-login branch October 18, 2023 16:44
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.

2 participants