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

fix: 대여소 위치 페이지 컴포넌트 에러 수정 #175

Merged
merged 1 commit into from
Oct 4, 2023

Conversation

ShinChanU
Copy link
Contributor

@ShinChanU ShinChanU commented Oct 1, 2023

PR Type

  • 버그수정 (Bugfix)

작업 내용

  • develop 브랜치에서 해당 페이지 런타임에러가 있길래 수정해서 올립니다.

Before

image

After

image

To Reviewers (참고 사항, 첨부 자료 등)

  • PR이 머지 되었을 때, 로컬에서 pull 받고 정상적으로 동작하는지 테스트를 한번 해보셨으면 하는 바람이 있습니다 ! 특히 충돌해결 후 머지나, 시간이 오래된 PR은 버전이 달라서 에러가 발생할 가능성이 높은 것 같아서요 !

  • 그리고 혹시 채연님, 지점 바텀시트에 대해서는 다른 팀 분들과 협의하신 부분이 있으실까요 ? 예전에 바텀시트를 드래그 올렸을 때, 대여소 소개 컴포넌트로 전환이 된다거나 그런 이슈가 있었던 걸로 기억해서요 ~
    @chaeyeon-yang

@ShinChanU ShinChanU self-assigned this Oct 1, 2023
Copy link
Member

@chaeyeon-yang chaeyeon-yang 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
Member

@heehminh heehminh 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

@LeeJSYS LeeJSYS left a comment

Choose a reason for hiding this comment

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

네 알겠습니다

@LeeJSYS LeeJSYS merged commit 43242df into UPbrella:develop Oct 4, 2023
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.

4 participants