Feature/fe-059 로그인 여부에 따른 기능제한 추가 #82
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2 tasks
jieunpark247
reviewed
Jul 5, 2023
| @@ -22,7 +22,7 @@ const LoginForm = () => { | |||
| { email, password, keep }, | |||
| { | |||
| onSuccess: () => { | |||
| router.replace('/'); | |||
| router.back(); | |||
There was a problem hiding this comment.
router.back()을 사용하는 경우에 로그인 페이지로 바로 진입시 먹팟을 벗어나는 이슈가 존재할 것 같아요!
보통 return url을 query param에 인코딩해서 넣고 replace 시켜주는 방법을 사용하는데, back으로 추가한 이유가 있을까요?
There was a problem hiding this comment.
그런 부작용이 있겠네요! 저는 기존 방식에서는 상세페이지에서 로그인 페이지로 갔을때 무조건 홈으로 가게되는 문제가 있어서 back 하는 방식을 적용해봤던거였는데 알려주신 방법을 적용하면 확실히 이탈을 막을수 있을 것 같네요 한번 적용해보도록 하겠습니다!
|
Kudos, SonarCloud Quality Gate passed!
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
비로그인 상태에서 먹팟 참여 버튼시 로그인 페이지로 이동
비로그인 상태에서 먹팟 만들기 클릭하면 로그인 페이지로 이동
체크 리스트
작업 내역
Profile쿼리를 미리 받아오는 로직이 많이 쓰여서Provider로 구성문제 상황과 해결
기존에는 로그인 성공 시 홈으로 이동하기로 구성되어있었으나 사용자 흐름상 더 자연스럽게 구성하기 위해 뒤로가기로 바꾸었습니다.
기존방식(무조건 홈으로 이동)
새로운 방식(뒤로 이동)
비고