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.

Sign up for GitHub

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

Jump to bottom

fix: sido 단위 마커를 클릭했을 때 동작 수정 #179

Closed
jaekwanAHN opened this issue Aug 8, 2023 · 0 comments · Fixed by #180
Closed

fix: sido 단위 마커를 클릭했을 때 동작 수정 #179

jaekwanAHN opened this issue Aug 8, 2023 · 0 comments · Fixed by #180
Assignees
@jaekwanAHN
Labels
bug Something isn't working

Comments

@jaekwanAHN
Copy link
Collaborator

작업 목록

  • 큰 틀
  • sido 단위 마커를 클릭했을 때 동작 수정
    • 작은 틀
    • sido 단위 마커를 클릭했을 때 sido에 속하는 구역들의 랭킹 페이지로 이동하던 현상을 수정합니다.
@jaekwanAHN jaekwanAHN added the bug Something isn't working label Aug 8, 2023
@jaekwanAHN jaekwanAHN self-assigned this Aug 8, 2023
@jaekwanAHN jaekwanAHN mentioned this issue Aug 8, 2023
Merged
tooooo1 pushed a commit that referenced this issue Aug 26, 2023
@jaekwanAHN
Fix: sido 단위 마크를 클릭하면 해당 지역 랭킹 페이지로 이동 (#180)
d3e959f 
## 📌 이슈번호 <!-- 이슈번호 혹은 참조를 적어주세요 -->

- close #179 

## 👩‍💻 요구 사항 <!-- 구현한 것을 간단하게 요약 , 코어 구현 로직 설명 -->
- 기존에 지도에서 sido 단위 마커를 클릭했을 때 `dust-forecast?sido=서울&city=서울` 과 같이 비정상적인
url로 이동하던 문제를 수정했습니다. 이제 sido 단위 마커를 클릭하면 `ranking/서울` 처럼 이동됩니다.

## 🎨 구현 스크린샷 <!-- .gif 등을 사용하여 간단하게 보여주세요 -->

[chrome-capture-2023-7-8.webm](https://github.com/tooooo1/dust-rating/assets/12118892/8f616a09-a162-46a6-be1b-bdb081d62fb7)

## ✅ PR 포인트 <!-- 리뷰어 분들이 집중적으로 보셨으면 하는 내용을 적어주세요 -->

## 질문 <!-- 궁금한 부분을 적어주세요 -->
- 클릭된 지역의 이름이 city라는 변수에 들어가 있기 때문에 sido 배열에서 city가 있다면 `ranking/city` 로
이동하도록 했는데 더 좋은 방법이 있을까요?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jaekwanAHN jaekwanAHN

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
1 participant
@jaekwanAHN