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

fix: 무한 설정 페이지 버그 수정합니다. #265

Merged
merged 4 commits into from
Aug 28, 2024
Merged

Conversation

summermong
Copy link
Collaborator

🤔 어떤 문제가 발생했나요?

  • 예를 들어 설정 -> 기록 시각화 기준 거리 수정 한 후 해당 화면에서 뒤로가기 버튼을 누르면 설정 페이지로 가게 됩니다. 여기서 또 뒤로가기 버튼을 누르면 기록 시각화 기준 거리 페이지로 나오게 되어 무한 반복됩니다. 따라서 일단 '/'로 수정했습니다.

🎉 어떻게 해결했나요?

📷 이미지 첨부 (Option)

⚠️ 유의할 점! (Option)

@summermong summermong added the fix # Fix bug label Aug 27, 2024
@king2dwellsang king2dwellsang added the Review Plz🙏 # Review is not yet complete label Aug 27, 2024
Copy link
Member

@Jungjjeong Jungjjeong left a comment

Choose a reason for hiding this comment

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

👍

@@ -48,7 +48,7 @@ export default function Page() {
<div>
<HeaderBar>
<HeaderBar.LeftContent>
<BackButton onClickBack={() => router.back()} />
<BackButton onClickBack={() => router.push('/')} />
Copy link
Member

Choose a reason for hiding this comment

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

기록 시각화 기준 거리 수정 후, 설정 페이지로 이동할 때 이벤트를 replace로 바꿔주시는건 어떨까요?

  • 홈 > 설정 > push > 기록 시각화 기준 거리 > replace > 설정
  • 결국은 홈 > 설정 > 설정의 히스토리만 남게 됩니다. (기록 시각화 기준 거리는 대체됩니다)

기록 상세에서도 동일한 흐름을 가져가고 있는데요, 요 방법으로 우선 적용해놓고 페이지 중복 문제는 한번에 해결해보시죠!

Copy link
Member

Choose a reason for hiding this comment

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

혹은 기준 거리 수정 후 > 뒤로가기 해도 괜찮겠네요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@Jungjjeong 좋은 의견 감사합니닷!!! 🙇🏻

@king2dwellsang king2dwellsang added Approved 🆗 # Review is completed and removed Review Plz🙏 # Review is not yet complete labels Aug 28, 2024
@summermong summermong merged commit 5efa663 into main Aug 28, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved 🆗 # Review is completed fix # Fix bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants