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

여행지 검색 필터 화면 개발 #55

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from

Conversation

Guboneui
Copy link
Owner

@Guboneui Guboneui commented Jan 8, 2024

PR은 내가 작성한 코드를 다른 사람이 보기 때문에 아래 사항을 최대한 구체적으로 작성 해주세요😘

1️⃣ 해당 PR을 생성하게 된 배경을 작성 해주세요

UI 구현의 경우, 피그마 이미지를 여기에 넣어주세요

  • TODO

2️⃣ 변경된 사항을 작성 해주세요

주요 변경 사항들에 대해 파일, 함수, 변수 등을 작성 해주세요.

  • TODO

3️⃣ 변경 사항에 대해 적용 방법을 공유해야 되는 경우(ex. DesignSystem) 코드를 통해 사용 방법을 공유 해주세요

없을 경우 작성하지 않아도 됩니다.

  • TODO

4️⃣ 신경 써서 봐야할 파일이 있다면 구체적으로 작성 해주세요

특정 파일 또는 코드에서 크로스 체크가 필요한 경우 작성 해주세요.

  • TODO

5️⃣ 결과를 작성 해주세요

UI의 경우 실제 화면 캡처 화면을, 추가 해주세요.
나머지 경우 'OOO 완료'로 작성 해주세요

  • TODO

컨플릭트 너무 많이 나서 차라리 롤백
@Guboneui Guboneui requested review from minneee and Donggaem January 8, 2024 17:37
@Guboneui Guboneui self-assigned this Jan 8, 2024
@Guboneui Guboneui changed the base branch from main to dev January 8, 2024 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants