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

전남대 Android_이가현 4주차 step2 과제 #74

Open
wants to merge 18 commits into
base: leeghy
Choose a base branch
from

Conversation

leeghy
Copy link

@leeghy leeghy commented Jul 19, 2024

스크린샷 2024-07-21 오전 10 22 46

스크린샷 2024-07-21 오전 10 41 27


  • 마커가 표시되지 않는데 해결하지 못했습니다. 남은 시간 더 해결해보도록 하겠습니다ㅠ
  • 테스트코드가 낯설어 얼만큼 작성해야하는지 감이 안잡혔습니다.

Copy link

@bigstark bigstark left a comment

Choose a reason for hiding this comment

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

가현님 고생 많으셨습니다!

조금 더 다음 스텝으로 나아가기 위해서는 기본적인 아키텍쳐에 대해 고민을 해보시는 것이 좋을 것 같아요.

또한 ViewModel 이나 Repository 등에 대한 테스트코드도 추가해주셔야할 것 같아요! 이 링크를 보신다면 조금 더 도움이 되지 않을까 싶네요!

Copy link

@bigstark bigstark left a comment

Choose a reason for hiding this comment

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

수고하셨어요!

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