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

Fix/#227 #235

Merged
merged 15 commits into from
Apr 15, 2024
Merged

Fix/#227 #235

merged 15 commits into from
Apr 15, 2024

Conversation

gnksbm
Copy link
Contributor

@gnksbm gnksbm commented Apr 15, 2024

작업내용

  • Tuist 위젯 관련 수정
  • 위젯 모듈 파일 구성, 경로 수정

리뷰요청

관련 이슈

close #227

@gnksbm gnksbm requested a review from isakatty April 15, 2024 09:19
@gnksbm gnksbm self-assigned this Apr 15, 2024
Copy link
Contributor

@isakatty isakatty left a comment

Choose a reason for hiding this comment

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

tuist widget 설정 외에도 위젯 코드들 확인했습니다 ! 고생하셨습니다 !!
주말에 공부하고 있긴 했어서 눈에 들어오는 키워드들이 조금 있긴한데 직접 만들어본게 아니라서 아직 명확하게 어떻다 이야기가 어렵네요ㅠ
잡아주신 기반을 바탕으로 위젯 개발해보겠습니다 !! 고생하셨어요 !

@@ -20,6 +20,7 @@ public extension String {
let date = Date()
let formatter = DateFormatter()
formatter.dateFormat = "yyyy.MM.dd.HH.mm"
formatter.locale = Locale(identifier: "ko_KR")
Copy link
Contributor

Choose a reason for hiding this comment

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

이 빌드 버전을 통해서 테스트 플라이트 올라갈 때 한국 시간으로 되는거군요 !
회의때 잠깐 이야기하고 지나갔는데 바로 적용,,,, 역시 , , 👍🏻

Button(intent: entry.configuration) {
Image(systemName: "arrow.clockwise")
}
.buttonStyle(.plain)
}
}
}

Copy link
Contributor

Choose a reason for hiding this comment

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

뷰를 어느 정도 만들어 놓으신건가용

Copy link
Contributor Author

Choose a reason for hiding this comment

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

위젯이 구분되도록 컴포넌트 올려둔거라 기존구현뷰는 지우고 작업하셔도 됩니다!

// An example configurable parameter.
@Parameter(title: "Favorite Emoji", default: "😃")
var favoriteEmoji: String
@Parameter(title: "정류장", optionsProvider: BusStopOptionProvider())
Copy link
Contributor

@yuhaeun-la yuhaeun-la Apr 15, 2024

Choose a reason for hiding this comment

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

BusOptionProvider() 로 사용자가 버스 번호를 동적으로 조절할 수 있게 만들어주는건가요?

Copy link
Contributor

Choose a reason for hiding this comment

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

DynamicOptionsProvider 이용해서 하는거군여

Copy link
Contributor Author

Choose a reason for hiding this comment

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

네 지원 타겟이 iOS 17.0이상이라 해당 방식을 선택했습니다

@isakatty
Copy link
Contributor

머지하겠습니다 !!~!

@isakatty isakatty merged commit c814b4d into dev Apr 15, 2024
1 check passed
@isakatty isakatty deleted the fix/#227 branch April 15, 2024 12:20
@yuhaeun-la yuhaeun-la added the ⚙ 환경세팅 환경세팅 label Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚙ 환경세팅 환경세팅
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] Widget 구성 변경
3 participants