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

🔀 :: (#519) 보관함 탭을 보관함과 내정보로 구분합니다 #520

Merged
merged 15 commits into from
May 8, 2024

Conversation

yongbeomkwak
Copy link
Member

@yongbeomkwak yongbeomkwak commented May 6, 2024

💡 배경 및 개요

  • 2.0에서 합쳐져있던 보관함 + 로그인 및 내정보를 3.0에서 별도의 탭으로 구분합니다.

Resolves: #519

📃 작업내용

  • MyInfo 퓨처 모듈을 만들어 MainTab에 추가해 줍니다.
  • SignIn기능이 다양한 화면에서 가능케 되기 때문에 interface를 통해 Factory로 접근하게 바꿨습니다.

🙋‍♂️ 리뷰노트

구현 시에 고민이었던 점들 혹은 특정 부분에 대한 의도가 있었다면 PR 리뷰의 이해를 돕기 위해 서술해주세요!

또한 리뷰어에게 특정 부분에 대한 집중 혹은 코멘트 혹은 질문을 요청하는 경우에 작성하면 좋아요!

e.g. 작업을 끝내야할 시간이 얼마 없어 확장성보다는 동작을 위주로 만들었어요! 감안하고 리뷰해주세요!

✅ PR 체크리스트

템플릿 체크리스트 말고도 추가적으로 필요한 체크리스트는 추가해주세요!

  • 이 작업으로 인해 변경이 필요한 문서가 변경되었나요? (e.g. XCConfig, 노션, README)
  • 이 작업을 하고나서 공유해야할 팀원들에게 공유되었나요? (e.g. "API 개발 완료됐어요", "XCConfig 값 추가되었어요")
  • 작업한 코드가 정상적으로 동작하나요?
  • Merge 대상 브랜치가 올바른가요?
  • PR과 관련 없는 작업이 있지는 않나요?

🎸 기타

Copy link

github-actions bot commented May 6, 2024

✅ Successful finished SwiftLint

Copy link

github-actions bot commented May 6, 2024

✅ 이슈와 PR의 Labels 동기화를 성공했어요!

@KangTaeHoon
Copy link
Contributor

아티스트 탭은 이미지가 없고, 차트 탭은 작아지네요.

@yongbeomkwak yongbeomkwak merged commit 8cd7ea3 into develop May 8, 2024
3 checks passed
@yongbeomkwak yongbeomkwak deleted the 519-split-tab-storage-myinfo branch May 8, 2024 08:40
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.

보관함 탭을 보관함과 내정보로 구분합니다.
4 participants