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

➕ [WV-38] feat : 익산 경기장 페이지 구현 #39

Merged
merged 11 commits into from
Jan 9, 2025

Conversation

Jinviz
Copy link
Member

@Jinviz Jinviz commented Jan 8, 2025

개요

wizpark/iksan에 있는 익산 경기장 페이지를 구현하였습니다.


PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • PR 제목 및 커밋 메시지 컨벤션 확인
  • 직접 만든 함수가 있다면 이에 대한 설명 추가 (ex. JS DOCS)
  • 변경 사항에 대한 테스트 완료 (버그 수정/기능에 대한 테스트)
  • Label 확인
  • Assignees 설정 확인
  • Reviewers 설정 확인

PR details

  • 기존 Map.tsx 컴포넌트를 common 폴더로 이동하여 위치에 따라 맵을 불러올 수 있도록 공통 컴포넌트화 하였습니다.
  • UI/UX 개선을 진행하였습니다.

개선 전

image

개선 후

image


When modifying code...

# Request Level
  - [ ] : "🔥 이대로 Merge 하면 안돼요~!"
  - [ ] : "🥹 고치면 분명 나아질 게 분명합니다.."
  - [ ] : "🤷 수정하면 좋지 않을까요?"

# Description

@Jinviz Jinviz added feat 새로운 기능 추가 design CSS 등 사용자 UI 변경 rename 파일 혹은 폴더명을 수정하거나 옮기는 작업만 하는 경우 labels Jan 8, 2025
@Jinviz Jinviz requested review from hee2323 and naelumcat January 8, 2025 13:48
@Jinviz Jinviz self-assigned this Jan 8, 2025
Copy link
Contributor

@hee2323 hee2323 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다

Copy link
Contributor

@naelumcat naelumcat left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.

Copy link
Contributor

@hayoung78 hayoung78 left a comment

Choose a reason for hiding this comment

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

확인했습니다

components/wizpark/iksan/bus-route-info.tsx Outdated Show resolved Hide resolved
components/wizpark/iksan/bus-route-info.tsx Outdated Show resolved Hide resolved
components/wizpark/iksan/bus-route-info.tsx Outdated Show resolved Hide resolved
@Jinviz Jinviz merged commit d9314e2 into dev Jan 9, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design CSS 등 사용자 UI 변경 feat 새로운 기능 추가 rename 파일 혹은 폴더명을 수정하거나 옮기는 작업만 하는 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants