Skip to content

[FEAT] 2.0 지역인증 UI (#134)#137

Merged
cirtuare merged 11 commits intodevelopfrom
feature/#134
May 13, 2025
Merged

[FEAT] 2.0 지역인증 UI (#134)#137
cirtuare merged 11 commits intodevelopfrom
feature/#134

Conversation

@cirtuare
Copy link
Contributor

🐿️ Pull Requests

🪵 작업 브랜치

🥔 작업 내용

  • 2.0 지역인증 UI 적용

🚨 참고 사항

네이버 Maps 사용으로 인한 변경사항

  • 지역인증 지도를 네이버 클라우드 API 중 디자인 스타일이 커스텀 가능한 Maps을 사용하게 되었습니다 !
  1. NMapsMap pod update : 3.19.0 -> 3.21.0
  • 올해 3월 말에 나온 맵 커스텀 프로퍼티인 customStyleID는 최신 버전에서 적용되어, 업데이트해주었습니다 ! 풀 받으신 후 pod repo update 해주세요
  1. Shared.xcconfig에 NMFNcpKeyID, NMFCustomStyleID 추가되었습니다 ! 파일 슬랙에 보내드릴게요 ~

📸 스크린샷

슬랙 디엠으로 영상으로 보내드릴게요 !

💥 To be sure

  • 모든 뷰가 잘 실행되는지 다시 한 번 체크해주세요 !

🌰 Resolve issue

@cirtuare cirtuare requested a review from yurim830 May 12, 2025 16:42
@cirtuare cirtuare self-assigned this May 12, 2025
@cirtuare cirtuare added 🌀 feature 새로운 기능 개발 🍓 수민 수민 labels May 12, 2025
Copy link
Collaborator

@yurim830 yurim830 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 변수 네이밍 부분 리뷰만 확인 부탁드려용

Comment on lines 35 to 36
// 🍇 TODO: 글모 Type 확인
let glassMorphismView = GlassmorphismView(.buttonGlassDisabled)
let glassMorphismView = GlassmorphismView(.gradientGlass)
Copy link
Collaborator

Choose a reason for hiding this comment

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

🐿️ 확인 됐으면 TODO 지워도 될 것 같아요!

Comment on lines 45 to 47
static var topInsetHeight: CGFloat {
return UIApplication.shared.windows.first?.safeAreaInsets.top ?? 0
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

🐿️ 네이밍이 조금 모호하게 느껴져서 safeAreaTopHeight 은 어떤지 제안드립니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

변경했습니다!

@cirtuare cirtuare merged commit 8181960 into develop May 13, 2025
@cirtuare cirtuare deleted the feature/#134 branch August 12, 2025 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🌀 feature 새로운 기능 개발 🍓 수민 수민

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] 2.0 지역인증 UI

2 participants

Comments