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

Exercise Javier Gonzalez #2

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

javglex
Copy link

@javglex javglex commented Feb 27, 2021

Hi Team,

In this project you will find I tried using SOLID principles combined with MVVM architecture recommended by Google. The UI requirements are INCOMPLETE as I couldn't find a good way to elegantly meet these requirements. However, I hope the rest of the projects show cases the readability, test-ability, and scalability requirements.

The entry point for the app is the MainActivity which inflates a navigation view. This navigation view loads the Manager's Specials fragment which contains a simple recycler view and grid layout.

The MainActivity also fires a repeating function that polls the network every 10 seconds. You will find in the comments I mentioned an alternative more realistic real world solution that doesn't rely on guessing best polling repeat speed.
I'm separating the project into app and core modules, core containing interfaces and models for our pure business logic. And everything else (presentation/framework) going inside app.
 
I created a simple repository that mocks the use of a local storage system, however more used for my own practice rather than to meet project requirements. All data is still fetched fresh from the network service every-time. 

Javier.Gonzalez added 4 commits February 19, 2021 18:59
Set up architecture to support SOLID principles.

Using latest architecture components for navigation, asynchronous calls, observables, DI, mvvm support, appcompat

In progress: exposing network status & dynamic display blocks
Saving Network response into a mock db just for personal practice.

Create network service for fetching managers specials

Added class that contains both canvas units and specials list
Removed unneeded scope from MainActivity
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.

1 participant