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

hotfix : 닫힘 버튼 #61

Merged
merged 4 commits into from
Sep 19, 2023
Merged

hotfix : 닫힘 버튼 #61

merged 4 commits into from
Sep 19, 2023

Conversation

seondal
Copy link
Member

@seondal seondal commented Sep 19, 2023

💡 왜 PR을 올렸나요?

💁 무엇이 어떻게 바뀌나요?

이전에는 상세페이지를 공유된 링크를 통해 바로 들어걀 경우 close 버튼을 눌러도 작동하지 않는 이슈가 있었습니다
(router.back()을 사용하는데 공유링크로 바로 들어가면 이전페이지가 존재하지 않음)
이에 router.back()을 없애고 로 변경하는 작업을 #60 에서 했더니 상세페이지에서 피드로 넘어갈 때 스크롤이 유지되지 않는 이슈가 있었습니다. 이에 이전페이지의 여부에 따라 분기 처리하여 작업이 수행되도록 했습니다

+) 파비콘 변경

💬 리뷰어분들께

빠른 어푸르브 부탁드려요!
작업 두번 시켜서 죄송합니다 8ㅁ8

@seondal seondal added PoseFeed 🧩 포즈피드 기능 관련 🛠 Fix 버그 수정 labels Sep 19, 2023
@seondal seondal added this to the 1st Deploy 🌟 milestone Sep 19, 2023
@seondal seondal self-assigned this Sep 19, 2023
Copy link
Collaborator

@guesung guesung left a comment

Choose a reason for hiding this comment

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

이제 확인했네요 ㅜㅜ

이전 페이지 여부를 document.referrer로 확인할 수 있군요 ! 👍 배워갑니당

@seondal seondal merged commit 754ae41 into develop Sep 19, 2023
@seondal seondal deleted the hotfix/maintain-scroll branch September 19, 2023 14:46
@seondal seondal added Detail 👀 상세페이지 and removed PoseFeed 🧩 포즈피드 기능 관련 labels Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Detail 👀 상세페이지 🛠 Fix 버그 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants