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

명칭 관련 논의 #12

Open
Pluu opened this issue Jan 28, 2020 · 1 comment
Open

명칭 관련 논의 #12

Pluu opened this issue Jan 28, 2020 · 1 comment
Assignees
Labels
question Further information is requested

Comments

@Pluu
Copy link
Member

Pluu commented Jan 28, 2020

Kind (Required)

  • Other

Overview (Required)

SessionDetailViewModel에서 사용하는 getSessionFromFirestore에 관한 질문입니다.

  • VM은 사용되는 데이터가 어디에서 오는지 알 필요가 없습니다.
  • Source의 정보를 노출함으로서 해당 스펙에 맞게 구현하게 되므로, 관심분리가 필요합니다.

Links

@Pluu Pluu added the question Further information is requested label Jan 28, 2020
@Jiyoung9310
Copy link
Member

아 로컬에도 데이터를 갖고 있다가 네트워크 상황이 좋지 않을 경우에는 내부 디비를 통해서 가져올 생각으로 작성했던건데, 생각해보니 뷰모델에서 구분할 것이 아니라 repo 내에서 알아서 데이터를 전달하게 끔 구현하는 것이 맞겠네요🤔 언급해주신 메소드명은 getSessionData 정도로 수정하면 좋을 것 같습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants