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

Refactor/home #12

Merged
merged 18 commits into from
Jun 3, 2024
Merged

Refactor/home #12

merged 18 commits into from
Jun 3, 2024

Conversation

ittzggd
Copy link
Collaborator

@ittzggd ittzggd commented May 3, 2024

이슈 (번호도 함께 작성해주세요)

해당 사항 (중복 선택)

  • FEAT : 새로운 기능 추가 및 개선
  • FIX : 기존 기능 수정 및 정상 동작을 위한 간단한 추가, 수정사항
  • BUG : 버그 수정
  • REFACTOR : 결과의 변경 없이 코드의 구조를 재조정
  • TEST : 테스트 코드 추가
  • DOCS : 코드가 아닌 문서를 수정한 경우
  • REMOVE : 파일을 삭제하는 작업만 수행
  • RENAME : 파일 또는 폴더명을 수정하거나 위치(경로)를 변경
  • ETC : 이외에 다른 경우 - 어떠한 사항인지 작성해주세요.

반영 브랜치

refactor/home -> dev
ex) feat/login -> dev

변경 사항

Home Tab과 관련된 뷰 및 비즈니스 로직에 대한 분리를 진행하였습니다

HomeView

  • CardView 내부의 CircularProgressBar 분리
  • 사용자 정보를 표시하는 HeaderView 분리
  • 특정 범위의 스크롤에 대한 새로고침을 진행할 수 있는 PullToRefresh 분리

HomeVM

  • 홈 내부 사용자의 주요 정보 및 체류시간 정보를 업데이트하는 비즈니스 로직을 분리하였습니다

그 외

  • 구조체 내부에 init()을 추가하였습니다

ex) 로그인 시, 구글 소셜 로그인 기능을 추가했습니다.

@ittzggd ittzggd requested a review from HiHoi May 3, 2024 15:13
@ittzggd ittzggd self-assigned this May 3, 2024
Copy link
Collaborator

@HiHoi HiHoi 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 62 to 66
#Preview {
@StateObject var homeVM = HomeVM()

return HeaderView(homeManager: homeVM)
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

오 이런 선언방식에 프리뷰도 사용이 가능하군요. 참고하겠습니다

@Binding var showNotice: Bool
var notice: Notice
var body: some View {
public var body: some View {
Copy link
Collaborator

Choose a reason for hiding this comment

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

혹시 public으로 설정하신 이유가 있을까요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

public으로 선언되어있는 NoticeModifier를 적용시키기 위함이었습니다!

HiHoi
HiHoi previously approved these changes May 21, 2024
Copy link
Collaborator

@HiHoi HiHoi left a comment

Choose a reason for hiding this comment

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

정말 수고하셨습니다!

Copy link
Collaborator

@HiHoi HiHoi left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@ittzggd ittzggd merged commit 247afaf into dev Jun 3, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants