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

[Feat] #27 - CustomAlertVC 구현 #28

Merged
merged 4 commits into from
Jan 4, 2023
Merged

[Feat] #27 - CustomAlertVC 구현 #28

merged 4 commits into from
Jan 4, 2023

Conversation

lsj8706
Copy link
Collaborator

@lsj8706 lsj8706 commented Jan 3, 2023

🌱 작업한 내용

  • CustomAlertVC 구현했습니다.
  • 지도에서 현 위치 오버레이의 아이콘을 디자이너분들이 만들어주신 에셋으로 변경했습니다. (기존의 파란색 -> 보라색)

🌱 PR Point

  • 혹시나 추후에 해당 AlertVC를 다른 곳에서 글자나 이미지만 바꿔서 사용할 수 있을 거 같아서 커스텀하여 구현했습니다.
  • 버튼의 터치 액션은 Combine framework 를 사용하여 받아오고 있는데 생소한 개념일 수 있기 때문에 혹시 지금 만든 CustomAlertVC를 사용하게 된다면 사용법을 물어보세요~! CourseDrawingVC에서의 사용을 참고해도 좋습니다!

📸 스크린샷

구현 내용 스크린샷
AlertVC
AlertVC 사용 예시 Simulator Screen Recording - iPhone 14 - 2023-01-04 at 03 29 18

📮 관련 이슈

@lsj8706 lsj8706 added Feat 새로운 기능 구현 세진☃️ labels Jan 3, 2023
@lsj8706 lsj8706 self-assigned this Jan 3, 2023
dlwogus0128
dlwogus0128 approved these changes Jan 4, 2023
View reviewed changes
Copy link
Contributor

@dlwogus0128 dlwogus0128 left a comment

Choose a reason for hiding this comment

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

새벽까지.. 고생이 많으시네욥.....🥹 저도 콤바인에 대해서 좀 찾아보겠습니다... 화이팅!

@lsj8706 lsj8706 merged commit b4ae9f3 into Runnect:develop Jan 4, 2023
@lsj8706 lsj8706 deleted the feat/#27-CustomAlertVC branch January 4, 2023 04:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dlwogus0128 dlwogus0128 dlwogus0128 approved these changes

@lee-yeonwoo lee-yeonwoo Awaiting requested review from lee-yeonwoo

Assignees

@lsj8706 lsj8706

Labels
Feat 새로운 기능 구현 세진☃️
Projects
Runnect-iOS
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feat] CustomAlertVC 구현
2 participants
@lsj8706 @dlwogus0128