feat: 포킷 모듈 기능 추가 및 수정 #182
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Shareable: 공유상태 추적용 프로토콜
포킷모듈의 링크편집로직에서 CategorySetting 로직이 필요한데 누락되어서 추가
- 기존 작성한 코드는 Public일 때 lock 아이콘을 보여주고 있었는데 반대로 작성해야 로직이 맞으므로 이에 따른 코드 수정
- fontsize 피그마 반영 - Spacer 최소너비 제거
- 2.7.4까지 PrivacyInfo file이 있는데 이후에 버전에서는 삭제가 되었음 from으로 설정하다보니 뒷버전이 실제 ci 구동시에는 2.7.6버전으로 받아와져서 수정함
7 tasks
ShapeKim98
approved these changes
Jan 15, 2025
Contributor
ShapeKim98
left a comment
ShapeKim98
left a comment
There was a problem hiding this comment.
수고하셨습다! CI 쪽 너무 힘드셨을텐데 고쳐주셔서 감사합니다.. 크게 별건 없네여 머지 하시면 될 것 같습니다~
| import Util | ||
|
|
||
| public struct BaseCategoryItem: Identifiable, Equatable, PokitSelectItem, PokitCardItem, Sortable { | ||
| public struct BaseCategoryItem: Identifiable, Equatable, PokitSelectItem, PokitCardItem, Sortable, Shareable { |
Contributor
There was a problem hiding this comment.
프로토콜이...ㅋㅋ POP 열심히 공부해서 개선해보겠습니다...
Member
Author
There was a problem hiding this comment.
인정합니다....
Sortable은 이제 필요가 없어졌고, PokitSelectItem과 PokitCardItem은 이름만 다르고 똑같던데 빨리돌아와주세요
| } | ||
|
|
||
| @MainActor | ||
| private var buttonLabel: some View { |
Contributor
There was a problem hiding this comment.
여기만 따로 @MainActor로 하신 이유가 있나여?
Member
Author
There was a problem hiding this comment.
수정: e551c4a
PokitCard 가져와서 건들다보니까 제거못한게 있었나봅니다 😅
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#️⃣연관된 이슈
#177
📝작업 내용
기능추가
수정
포킷카드 디자인 수정
공유 여부에 따른 자물쇠 아이콘과 공유한 인원 수를 보여주기 위해 디자인을 수정했습니다.
미분류 링크 편집하기에서 포킷 이동 시 카테고리 목록에
미분류 카테고리가 남아있던 버그 수정tuist 의존성 수정 (FeatureCategorySetting 추가)
이번에 편집하기 기능에서 import해서 사용하던 모듈인데 이때문에 ci도 에러가 났었습니다. 깜빡하고 추가못해서 수정합니다..
build_test 에러 수정
2.7.6버전을 받는 것을 확인했는데 해당 버전은 PrivacyInfo 파일이 제거된 버전입니다. 제 생각으로는 캐시로 남아있는 2.7.4버전에서 ci시 나오는 2.7.6버전의 SwiftSoup로부터 PrivacyInfo파일을 찾는 도중 에러가 발생한 것으로 짐작하고 있습니다.
16.2이며 18.2버전의 시뮬레이터를 찾을 수 없다고 지속적으로 에러를 뱉는 상황입니다. 임시방편으로 문제 해결시까지 xcode16.1버전을 사용할 예정입니다.스크린샷 (선택)
포킷 카드 수정 & 즐겨찾기 카드포킷이동 메세지 추가close #177