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]: 리팩토링 #6

Open
wants to merge 3 commits into
base: develop/view
Choose a base branch
from
Open

[REFACTOR]: 리팩토링 #6

wants to merge 3 commits into from

Conversation

0zlrlo
Copy link
Member

@0zlrlo 0zlrlo commented Dec 21, 2022

No description provided.

@0zlrlo 0zlrlo requested a review from b1urrrr December 21, 2022 08:14
Copy link
Member

@b1urrrr b1urrrr left a comment

Choose a reason for hiding this comment

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

오류 고치고 패키지 분리까지 하느라 고생 많으셨어요😭
패키지 분리만 제대로 해도 작업할 때 훨씬 편할 거예요 ㅎㅎ
화이팅!!

@@ -1,9 +1,11 @@
package org.sopt.sample
package org.sopt.sample.presentation.activity
Copy link
Member

Choose a reason for hiding this comment

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

presentation 패키지를 activity, adapter, fragment로 분리하셨네요!
저희 앱잼 때 사용할 방식처럼 뷰마다 패키지를 만들어보면 좋을 것 같아요~
(제 레포지토리 참고하셔도 좋을 것 같아요 🙃)

EX) presentation 안에 signin, signup, main

Copy link
Member Author

Choose a reason for hiding this comment

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

넵 감사합니다!

@@ -1,9 +1,11 @@
package org.sopt.sample
package org.sopt.sample.presentation.activity
Copy link
Member

Choose a reason for hiding this comment

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

HomeActivity는 지금은 필요하지 않는 뷰 같은데 관련된 파일들을 삭제하면 더 깔끔할 것 같아요!

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