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

Merging changes of Week 1 through 4 via final branch #3

Merged
merged 14 commits into from
Jan 6, 2022
Merged

Conversation

pk-218
Copy link
Member

@pk-218 pk-218 commented Jan 6, 2022

Changes made since week-4 commit:

  • Added Splash Screen - updated theme in AndroidManifest, changed all Activity screens to show the splash screen, added splash.xml, and corresponding drawable to be displayed.

  • Refactored Article from Response.kt for RecyclerView model.

  • Added Bookmark icon to ActionBar to navigate to BookmarksFragment

    image

  • Added Bookmark entity, BookmarkDao interface, BookmarkRoomDatabase.

  • Added NewsRepository as a layer between ListViewModel and data observed from RoomDB and Retrofit service. Modified ListViewModel with Room logic and added ViewModelFactory.

  • Modified NewsFragment to show web view of a selected news article with floating action button to bookmark it.

  • Updated ListFragment and RecyclerViewAdapter to show divider.

@pk-218 pk-218 requested a review from sans2801 January 6, 2022 12:15
@sans2801 sans2801 merged commit 78e5ffd into main Jan 6, 2022
@sans2801
Copy link
Collaborator

sans2801 commented Jan 6, 2022

Thanks

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.

3 participants