-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: [2.0.0] 버전 업데이트에 따른 기능 추가 및 수정(포킷 수정 추가) #186
Conversation
- 포킷 리스트를 띄우는 로직 제거 - domain 내 categoryListInQuiry 프로퍼티 제거
- 전체공개 설정 섹션 구성 - 포킷 키워드 섹션 구성 - 프로필 background Circle 및 수정 아이콘 padding값 수정
- 공유 여부에 따라 화면 구성이 달라지는 형태로 바뀌기 때문에 새로 구성
- v2 api에 따른 openType, keywordType 프로퍼티가 추가된 request값 반영
- v2 api에 따른 domain값 수정
- 키워드 선택값에 따라 보여줘야 하는 텍스트 구성과 컬러값이 달라 추가함 - default상태 및 초기 상태는 normal이며, 카테고리 공개를 한 상태에서 저장을 눌렀을 때 키워드가 입력이 되지 않는다면 warnning상태로 바뀜
- isFavorite field 추가로 인한 DTO, Domain 수정
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다! 전반적인 코드에선 딱히 문제 없어보이구 그 질문 남겨주신 시트 높이 관련해서 좀 디버깅을 해보았는데,
...
.ignoresSafeArea(edges: [.bottom, .top])
.readHeight()
.onPreferenceChange(HeightPreferenceKey.self) { height in
if let height {
self.height = height
}
}
.presentationDetents([.height(height)])
ignoreSafeArea
모디파이어를 readHeight()
보다 먼저 두는걸로 어느정도 해결이 된거 같은데, 시트를 위로 잡아 당기면 다시 safe area 포함해서 높이가 변경되는 문제는 여전한거 같습니다.
한가지 의심 가는점이, #185 여기서 PokitFlowLayout
이 잘못되어 수정한 코드를 올렸었는데, 이 잘못된 레이아웃이 영향이 있지않을까..? 란 의심입니다. 한번 머지 해보고 다시 확인해보져!
#️⃣연관된 이슈
#181
📝작업 내용
포킷 추가 및 수정할 때 화면 처리와 추가된 기능들에 대해 작업했습니다.
전체공개시에는 키워드를 반드시 체크해야 하며, 비공개시 키워드는 입력하지 않아도 됩니다.
이에따라 포킷수정/추가, 카테고리 공유 후 추가 api도 v2에 맞게 작업하였으며 domain 및 일부 DTO가 수정되었습니다.
스크린샷 (선택)
전체공개O, 키워드선택
키워드 미선택 로직
💬리뷰 요구사항(선택)
시트 높이가 컨텐츠 높이만큼 받아지지가 않아서 제가 전에 수정했었는데 이것도 적용이 안되는 것 같더라구요. safearea 관련해서 열심히 찾아보고 있습니다. 아시는 부분 있으시면 공유해주세요 🥲
close #181