Skip to content

Conversation

@zzzRYT
Copy link
Contributor

@zzzRYT zzzRYT commented Dec 7, 2025

🔗 관련 이슈

#197

📝작업 내용

  • 각 숏폼이 id형태로 받아오게 하기 이전에, filter되던 값을 스토리지로 변경
스크린샷 2025-12-07 오후 10 34 20 - url에 id가 저장될 수 있도록 router 추가 (아직 동작하진 않음)

🔍 변경 사항

  • tour fiter에 대한 정보를 url의 query-string형태로 저장했었는데 스토리지에 저장하도록 변경
  • 변경 사항 2
  • 변경 사항 3

💬리뷰 요구사항 (선택사항)

  • 파일체인지가 많은데, 그냥 단순히 url의 정보를 useLocalStorage를 통해서 불러올 수 있도록 바꾼것 뿐이라 가볍게 봐주시면 좋을 것 같습니다. (실제론 변경된거 많지 않음)

@zzzRYT zzzRYT requested a review from electrohyun December 7, 2025 13:30
Copy link
Contributor

@rhehfl rhehfl left a comment

Choose a reason for hiding this comment

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

@zzzRYT 이슈와 PR 확인했습니다.
url로 바로 접속 시 관광지 위치를 토대로 작동하도록 하는건 좋은 것 같습니다!
개인적으로는
image
이 부분분이 로직이 까다롭고 버그가 잦을 것 같은데 아얘 탭을 분리해보는건 어떨까요?

image url부분은 좀 더 생각해봐야 할것같은데 이런식으로 다른 탭으로 하면 어떨까.. 생각이 들었ㅅ브니다.

@zzzRYT
Copy link
Contributor Author

zzzRYT commented Dec 8, 2025

@rhehfl 숏폼의 특성상 콘텐츠가 끊기지 않도록 해야할 것 같습니다.

그래서 마지막에 렌더링한 요소를 기준으로 다시 재 렌더링시켜서 하자고 제안한건데, 생각해보면 이 전 바운더리랑 겹치면 중복된 컨텐츠가 나올 수 있어서 별로 안좋겠구나 싶긴 하네요,

그래도 계속 콘텐츠가 이어지도록 하려면 마지막 기준으로 해야할 것 같긴 합니다. 아니면 컨텐츠 타입을 바꿔서 하게 만들던가 그런식으로 구성해야 할 것 같네요

@zzzRYT
Copy link
Contributor Author

zzzRYT commented Dec 8, 2025

@rhehfl 위에서 언급해 주신 내용에 대해서는, 하나의 단일 페이지가 좋을 것 같습니다. 차라리,내 주변으로 탐색하기 탭을 side에 넣어서 토글시킬 수 있도록 하는 방향을 생각하고 있습니다.

근데 이게 좀 힘든게 기존의 page형식을 전부 버리고, id형식으로 바꾸는거다 보니 변경하는 부분에서 어려운점이 있어서 작업이 오래걸릴 것 같습니다.

@zzzRYT zzzRYT closed this Dec 8, 2025
@zzzRYT zzzRYT reopened this Dec 8, 2025
@zzzRYT zzzRYT merged commit 9f56812 into base/#197/shorts Dec 9, 2025
3 checks passed
@zzzRYT zzzRYT deleted the modify/#197/shorts-1 branch December 9, 2025 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants