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

Feature/view - Update View & Template #48

Merged
merged 6 commits into from
Nov 9, 2023
Merged

Feature/view - Update View & Template #48

merged 6 commits into from
Nov 9, 2023

Conversation

srlee056
Copy link
Contributor

@srlee056 srlee056 commented Nov 9, 2023

  • 전국 / 시도 기준으로 소속된 지역들을 리스트로 보여주는 부분 구현

    • 전국 페이지 : 전국의 시/도 리스트
    • ㅇㅇ시 페이지 : ㅇㅇ시에 포함된 구/군 리스트

  • 각 지역들은 a tag를 통해 링크가 걸려있으며, 링크는 query 형태로 구현되어있음

    • ex) 서울특별시 : ?sido_nm=서울특별시
    • ex) (서울특별시) 강서구 : ?sido_nm=서울특별시&gugun_nm=강서구

yeeeeeeen700
yeeeeeeen700 reviewed Nov 9, 2023
View reviewed changes
Copy link
Contributor

@yeeeeeeen700 yeeeeeeen700 left a comment

Choose a reason for hiding this comment

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

시/구/동 분류 확인했습니다!

@yeeeeeeen700 yeeeeeeen700 merged commit b18521e into main Nov 9, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yeeeeeeen700 yeeeeeeen700 yeeeeeeen700 left review comments

@jeremy714 jeremy714 Awaiting requested review from jeremy714

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@srlee056 @yeeeeeeen700