Fixes #49: Added Firebase SDK

dynamite8 / resources-android-dev

Collection of resources for becoming an Android developer

7 stars 7 forks source link

Fixes #49: Added Firebase SDK #50

Closed dynamite8 closed 6 years ago

dynamite8 commented 6 years ago

Fixes # 49.

Changes proposed in this pull request:

Added Firebase SDK dependency
Removed some unused imports

JaeW commented 6 years ago

@dynamite8 If we're going to use Firebase I would suggest that the data be pulled in through the Repository instead of the UI. This makes testing easier and conforms to the MVVM architecture model.

JaeW commented 6 years ago

@oa2013 Actually I don't want to merge it in as @dynamite8 and I are stilling trying to determine if Firebase is the right thing to use as the backend. Even if it is, instead of the data being pulled in through the UI (Activity or Fragment module), it should be pulled in through the repository so that the UI won't be affected by the data source - even if that should change.