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

[Feat/#114] Shared Cell Feature로 이동 #115

Merged
merged 4 commits into from
Jul 3, 2023

Conversation

derrickkim0109
Copy link
Collaborator

PR 요약

📌 변경 사항

  • Shared에 있던 Profile, Crew 내의 Cell Feature로 이동
  • Profile 좋아요 버튼 이미지 변경
✅ PR check list
  • id 넘기는 걸 어떻게 가정하고 넘겨야 할지 잘 모르겠음. 홈 뷰의 리스트에서 각 셀의 userID값을 가지고 있고 내 userID값을 가지고 있어서 그걸 비교해서 처리해야 하는 건가?

Linked Issue

close #114

# Conflicts:
#	Projects/Feature/Crew/Interface/Sources/Root/CrewRootStoreInterface.swift
Copy link
Member

@mooyoung2309 mooyoung2309 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 !

원래 있던 파일들을 옮긴 작업인건가요 ?!

id 넘긴다는 것은 다음 화면으로 넘어갈때 id 를 넘기는 방식을 얘기하는 건가요 ?
제쪽 코드에서 그런상황들이 많아요. 다음 화면으로 의존성을 넘기는 상황이 많으니깐 참고하시면 댈것같아요. featureWorkoutHome -> featureWorkoutStart -> featureWorkoutTimer 같은 데이터를 2번 넘기는 경우도 있습니다.

@derrickkim0109
Copy link
Collaborator Author

응! 맞아

@mooyoung2309
Copy link
Member

머지 시켜주시면 될것같아요 !

@mooyoung2309 mooyoung2309 merged commit 1e28667 into develop Jul 3, 2023
@mooyoung2309 mooyoung2309 deleted the feat/#114-ProfileUI-Refactoring branch July 3, 2023 00:59
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.

[feat] Profile Cell Shared에서 Feature로 이동
2 participants