-
Notifications
You must be signed in to change notification settings - Fork 2
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
Refactor/home #12
Refactor/home #12
Conversation
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.
전체적으로 확인했습니다. 코멘트 확인 부탁드립니다. 수고하셨습니다!
HANE24/View/Home/HeaderView.swift
Outdated
#Preview { | ||
@StateObject var homeVM = HomeVM() | ||
|
||
return HeaderView(homeManager: homeVM) | ||
} |
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.
오 이런 선언방식에 프리뷰도 사용이 가능하군요. 참고하겠습니다
HANE24/View/Home/NoticeView.swift
Outdated
@Binding var showNotice: Bool | ||
var notice: Notice | ||
var body: some View { | ||
public var body: some View { |
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.
혹시 public으로 설정하신 이유가 있을까요?
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.
public으로 선언되어있는 NoticeModifier를 적용시키기 위함이었습니다!
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.
정말 수고하셨습니다!
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.
수고하셨습니다!
이슈 (번호도 함께 작성해주세요)
해당 사항 (중복 선택)
반영 브랜치
refactor/home -> dev
ex) feat/login -> dev
변경 사항
Home Tab과 관련된 뷰 및 비즈니스 로직에 대한 분리를 진행하였습니다
HomeView
HomeVM
그 외
ex) 로그인 시, 구글 소셜 로그인 기능을 추가했습니다.