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

✨ 아이폰 레이아웃 라이브러리 적용 #83

Merged
merged 4 commits into from
May 29, 2024

Conversation

BangDori
Copy link
Collaborator

작업 이유

  • 기존 코드 베이스의 복잡성을 줄이고자, 아이폰 레이아웃 라이브러리를 추가하였습니다.

작업 사항

  • 기존 아이폰 레이아웃을 담당하는 소스 제거
  • 아이폰 라이브러리 추가

리뷰어가 중점적으로 확인해야 하는 부분

  • 현재 아이폰 레이아웃을 관리하는 라이브러리가 devDependencies가 아닌 dependencies에 포함되어 있습니다. 실제 서비스를 배포하는 시점부터는 아이폰 레이아웃이 필요없게 되니, -D 옵션으로 처리한 후 개발 환경에서만 동작하도록 수정하면 되겠습니다.

발견한 이슈

  • 없음

@BangDori BangDori self-assigned this May 29, 2024
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-83.d37mn03xh3qyyz.amplifyapp.com

Copy link
Member

@Legitgoons Legitgoons left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 병합하셔도 좋습니다!

@BangDori BangDori merged commit e30bc1d into main May 29, 2024
2 checks passed
@BangDori BangDori deleted the feature/react-iphone-layout branch May 29, 2024 11:48
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.

2 participants