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

NY Times books #8

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

NY Times books #8

wants to merge 4 commits into from

Conversation

gorrotowi
Copy link

Some UI idea to build

Captura de pantalla 2024-07-25 a la(s) 4 22 42 p m

Actual state

Screenshot_1721946686

What is pending

  • Implement a data source (like Room) to store the last response to return data when the user doesn't a internet connection
  • Implement merge data in the repository to retrieve just the ResultData object
  • Change layout orientation in home RecyclerView when you hit the toggle view
  • Add a swipeToRefresh or a button to re-fetch the book list from the ViewModel
  • Add loading and error message in the screen to inform the user when the service/data is not available
  • Fix ViewModel and repository Tests
  • Inject dependencies like API-KEY to build network client

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