Don Turner
ad1ec28eb3
Merge pull request #678 from SimonMarquis/patch-2
...
Don't rely on `finalizeDsl` API to configure Crashlytics
2 years ago
Takeshi Hagikura
b3cdc172cd
Implement search feature ( #685 )
...
Implement search feature
- Add a feature module named "search"
- Add a SearchScreen that is navigated by tapping the search icon at the top left corner
- Add a data layer that takes care of populating the *Fts tables and querying them by a search query
- Add a SearchViewModel that wires up the data layer of the Fts tables with the SearchScreen
The SearchScreen has following features:
- The user is able to type the search query in the TextField
- The search result is displayed as the user types
- When the search result is clicked, it navigates to:
- The InterestsScreen when a topic is clicked
- Chrome custom tab with the URL of the clicked news resource
- When the search result is clicked or the IME is explicitly closed by the user, the current search query in the TextField is saved as recent searches
- Latest recent searches are displayed in the SearchScreen
2 years ago
Adetunji Dahunsi
73a38720d8
Merge pull request #649 from android/tj/backend-requested-sync
...
Wire up backend requested sync
2 years ago
Adetunji Dahunsi
6b834b6f4c
Applied code review suggestions
...
Change-Id: I12fab8e0d27a8a1805e0063054ec3382bebd78f1
2 years ago
Adetunji Dahunsi
b3f2502ec2
Update core/data/src/test/java/com/google/samples/apps/nowinandroid/core/data/repository/OfflineFirstNewsRepositoryTest.kt
...
Co-authored-by: Don Turner <dturner@users.noreply.github.com>
2 years ago
Adetunji Dahunsi
f86d174c1b
Used more explicit variable names in tests
...
Change-Id: I2e0ebead70441eced05eb786aaa7683bfefa8976
2 years ago
Adetunji Dahunsi
337c940d66
Add comments explaining backend sync and made variable names easier to grok
...
Change-Id: I90cd7444de95efa20bf243a922a772f7849a23ec
2 years ago
Adetunji Dahunsi
022cd92f4c
Update core/data/src/main/java/com/google/samples/apps/nowinandroid/core/data/repository/OfflineFirstNewsRepository.kt
...
Co-authored-by: Don Turner <dturner@users.noreply.github.com>
2 years ago
Neelansh Sahai
adc9381bb5
Merge pull request #640 from android/fr-undo-bookmark-removal
...
Add Undo snackbar on Bookmark removal
2 years ago
Milosz Moczkowski
6d87dc6888
Merge pull request #680 from SimonMarquis/patch-4
...
Delete no longer used `AUTHORS_ASSET` reference
2 years ago
Simon Marquis
9499e559e7
Delete no longer used `AUTHORS_ASSET` reference
2 years ago
Simon Marquis
80b8ae7a98
Don't rely on `finalizeDsl` API to configure Crashlytics
2 years ago
Neelansh Sahai
bf747434cd
Add Undo snackbar on Bookmark removal
...
Change-Id: I1fefd6e72378e26ae35b66e032529a116cff9a79
2 years ago
Milosz Moczkowski
2c18740d62
Merge pull request #665 from AnirudhPudari/migrate-to-compose_flowlayouts
...
[Bug]: Migrate away from accompanist/flowlayout
2 years ago
Milosz Moczkowski
6bf37adde9
Merge pull request #671 from android/kotlin-version-updates
...
Update Kotlin, Compose compiler, serialization versions
2 years ago
James Rose
3e66e3d7e8
Merge pull request #595 from android/jr/track-viewed
...
Add visual indicator of read/unread news resources
2 years ago
Márton Braun
bea2eecd90
Update Kotlin, Compose compiler, serialization versions
...
Change-Id: I2aa0d07769e79e6f85824a391418566aef6b721e
2 years ago
Adetunji Dahunsi
05be2855d8
Wire up backend requested sync
...
Change-Id: I1d4485b589c7e94527a2a02f371cd3f030231622
2 years ago
James Rose
93953c2206
Add missing method doc
2 years ago
James Rose
050db2cb72
Merge branch 'main' into jr/track-viewed
2 years ago
James Rose
b49767a88c
Rename getUserNewsResources to observeAll
2 years ago
AnirudhPudari
33a3e360a4
Deprecated FlowLayouts from accompanist
2 years ago
Milosz Moczkowski
feafb5fbed
Merge pull request #575 from takagimeow/fix/provide-true-to-local-inspection-mode
...
Provide true to LocalInspectionMode
2 years ago
Milosz Moczkowski
2b2f6d48ce
Merge pull request #600 from takagimeow/cleanup-unused-args
...
Remove newsResourceTitle from AnalyticsHelper.logNewsResourceOpened method
2 years ago
Milosz Moczkowski
37908ec05c
Merge pull request #630 from drawers/david/delete-unnecessary-gradle-wrapper
...
Remove unnecessary copy of Gradle wrapper
2 years ago
Adetunji Dahunsi
67dae0c3c0
Merge pull request #650 from android/tj/manual-sync
...
Initialize sync manually
2 years ago
Adetunji Dahunsi
489502ba13
Initialize sync manually
...
Change-Id: Ia26b1179270a87d3b87a915185b8d5fc7532c464
2 years ago
Milosz Moczkowski
f3faec8432
Merge pull request #643 from android/tm/report-drawn
...
Use ReportDrawnWhen
2 years ago
mlykotom
3b7e7e3c02
Use ReportDrawnWhen
...
Change-Id: I77b4db63f72f43c6c826e66358f7672792ef137d
2 years ago
Milosz Moczkowski
452ab50e98
Merge pull request #449 from SimonMarquis/patch-2
...
Configure `isReturnDefaultValues` to fix #448
2 years ago
Milosz Moczkowski
0b0fdbfdbf
Merge branch 'main' into patch-2
2 years ago
Adetunji Dahunsi
08a165cf6a
Merge pull request #631 from android/tj/notifications
...
Backend triggered sync
2 years ago
Adetunji Dahunsi
08956492c8
Backend triggered sync
...
Change-Id: I53c43b136ebb755f6258b1e815301dddb3b536a3
2 years ago
Milosz Moczkowski
ba6a697844
Merge pull request #624 from SimonMarquis/gradle-sort
...
Sort and group Gradle dependencies
2 years ago
Milosz Moczkowski
1ab9de2bd0
Merge pull request #470 from c5inco/ui-polish
...
[UI Polish] Adjusted spacing based on design spec
2 years ago
Chris Sinco
93c01ad184
Fix trailing commas based on spotless check
2 years ago
Chris Sinco
049750dd51
Merge branch 'main' into ui-polish
2 years ago
David Rawson
973eb04d81
Remove unnecessary Gradle wrapper
2 years ago
Alex Vanyo
2388d4eb24
Merge pull request #626 from amaurymedeiros/patch-1
...
Fix ForYouScreenTopicSelection Compose Preview
2 years ago
Amaury Medeiros
6b605a779c
Fix ForYouScreenTopicSelection Compose Preview
2 years ago
Don Turner
1f7fc94082
Merge pull request #625 from SimonMarquis/patch-3
...
Remove debug logs from CI workflow
2 years ago
Simon Marquis
e5d29547a1
Remove debug logs from CI workflow
...
Closes #584
2 years ago
James Rose
57c13d84bd
Incorporate code review changes: Move UserNewsResourceRepository to data
...
module; move UserNewsResource to model module. Implement unread dot for
bookmarked articles. Keep the flows cold in UserNewsResourceRepository.
2 years ago
Simon Marquis
8ec54d19d8
Sort and group Gradle dependencies thanks to square/gradle-dependencies-sorter
2 years ago
James Rose
ebfbb5bafd
Display unread state on the news feed and bottom nav bar
...
When a news resource is unread, display a dot on its card in the news
feed. When the For You section has unread resources, display a dot on
its icon in the navigation bar.
Update the read status when a resource is opened.
2 years ago
James Rose
bd450099fb
Replace GetUserNewsResourcesUseCase with UserNewsResourceRepository
...
This moves the responsibility for joining the UserData and the
NewsResources to UserNewsResourceRepository. This way, the work can be
done once and shared with all consumers in a SharedFlow, rather than
having each consumer perform the join itself by invoking the UseCase.
2 years ago
James Rose
9ee2acf327
Add viewed status for news resources to data layer
2 years ago
Adetunji Dahunsi
ebdb8589a6
Merge pull request #608 from android/tunjid-batch-sync
...
Batch sync news resources from remote
2 years ago
Don Turner
6d1f85b8ce
Merge pull request #607 from android/mv/tests
...
Provide a CoroutineScope to fake DataStores
2 years ago
Adetunji Dahunsi
8765d3ac57
Fix tests for batched news resource sync
...
Change-Id: I37545eba8618edc936eb12fa38628d98f85f52ed
2 years ago