nowinandroid

Commit Graph

Author	SHA1	Message	Date
TJ Dahunsi	ce6eaa9d64	Fix spotless	1 year ago
TJ Dahunsi	4858167b24	Added clarifying comments to code and used better variable names	1 year ago
TJ Dahunsi	bda31e9d15	Default scrollbars to dormant	1 year ago
TJ Dahunsi	d6a265093d	Extract scrollbar values to constants	1 year ago
TJ Dahunsi	795c5e32a9	Add hoverable to scrollbar	1 year ago
TJ Dahunsi	31b4841cb2	Add scrollbars to app	1 year ago
Márton Braun	56389cb760	Merge pull request #752 from SimonMarquis/kotlin-1.9.0 Kotlin 1.9.0	1 year ago
Simon Marquis	f7ed38182d	Add protobuf generated sources to the sourceSets And update to version 3.23.4.	1 year ago
blackbracken	b570be2abb	reformat by spotless	1 year ago
blackbracken	e9a16e437d	add kdoc for getNewsResourceIds	1 year ago
blackbracken	805914dade	Call getNewsResourceIds instead of getNewsResources	1 year ago
Milosz Moczkowski	7b6c3a16ca	Merge pull request #823 from ahmed-madhoun1/allow_localised_date_format Allow localised date format	1 year ago
Don Turner	b2a74ba174	Merge pull request #812 from Askeri-Muhendis/main Kotlin Coding conventions	1 year ago
TJ Dahunsi	c4686cfa54	Track when deep link notifications are opened Change-Id: I223b1c8fbca36f9b7f2a35d9cb2fa901d6f5be91	1 year ago
ahmedmadhoun1	3114b2eeaa	Allow localised date format	1 year ago
İbrahim Ethem Şen	6271d1c4c0	Merge branch 'android:main' into main	1 year ago
İbrahim Ethem Şen	56f8a27132	Kotlin coding conventions places companion object at the bottom of the class	1 year ago
Tomáš Mlynarič	8e1e994bb1	Add named arguments Change-Id: Ic91ae4fc04bf8eb92c3e5845c7e984475526ec91	1 year ago
Tomáš Mlynarič	4a8a43551a	Add contentType to news feed Before ScrollForYouFeedBenchmark_scrollFeedCompilationFull frameDurationCpuMs P50 4.9, P90 6.9, P95 7.6, P99 9.5 frameOverrunMs P50 -10.3, P90 -6.8, P95 -2.8, P99 -2.1 Traces: Iteration 0 1 2 3 4 5 6 7 8 9 After ScrollForYouFeedBenchmark_scrollFeedCompilationFull frameDurationCpuMs P50 4.6, P90 6.3, P95 7.2, P99 9.2 frameOverrunMs P50 -10.6, P90 -8.7, P95 -7.8, P99 -4.7 Traces: Iteration 0 1 2 3 4 5 6 7 8 9 Change-Id: I5589776f9627c27154873529c9ca9dd055a8a8ef	1 year ago
James Rose	7826b4f5f2	Merge pull request #748 from SimonMarquis/fix/permissions Grant `POST_NOTIFICATIONS` permission in more instrumented tests	1 year ago
Adetunji Dahunsi	890c0f5769	Merge pull request #776 from SimonMarquis/fix/714 Keep track of matching `Network`s inside `NetworkCallback`	1 year ago
Simon Marquis	27154ef38a	Apply changes from review comments	1 year ago
Miłosz Moczkowski	479a2f00b0	Refactor icons	1 year ago
Milosz Moczkowski	1345764649	Merge pull request #770 from android/refactor/remove_unused_statements Remove unused statements	1 year ago
Simon Marquis	fecab96e4c	Keep track of matching `Network`s inside `NetworkCallback` This will ensure the connectivity state remains synchronized with the `ConnectivityManager`. Fixes #714	1 year ago
Miłosz Moczkowski	99094199bc	Remove redundant suppressions Change-Id: Iae08b6f284cf7a387922fdec34c8a0aa0da809ef	1 year ago
Miłosz Moczkowski	5bf66739bd	Remove redundant statements Change-Id: Iaa25b7ba033d5e9c85cc2dc433197ed7b8b6cb89	1 year ago
Simon Marquis	9aba5cc0fe	Cleanup unused string resources	1 year ago
Adetunji Dahunsi	f914254414	Merge pull request #749 from android/tj/reactive-only-dao Ensure DAO exposes only reactive types	1 year ago
Don Turner	d319264ef0	Provide Default dispatcher for instrumented tests Change-Id: I488a9b9d9d3864ce9496614ab35332ec19d06bcc	1 year ago
Don Turner	b245334a18	Updating TestDataStoreModule to use ApplicationScope	1 year ago
TJ Dahunsi	dcc36b0228	Fix tests Change-Id: Ie6b62071d2023e3a7ffe1553ab606e3f6b386a5e	1 year ago
Don Turner	4cdb5963a0	Add top level application scope, use IO dispatcher for DataStore Change-Id: I1512b1665587c73abd5e8a78aa9abd3eed24ab79	1 year ago
TJ Dahunsi	cd6ad7d575	Ensure DAO exposes only reactive types Change-Id: I8992ccf6525cd4c5ea9503d532b500e4d310b7fb	1 year ago
Simon Marquis	823c4db201	Grant `POST_NOTIFICATIONS` permission in more instrumented tests Continues the work initiated in #738. Extract the SDK version check inside a `GrantPostNotificationPermissionRule` class that delegates to a regular `GrantPermissionRule`.	1 year ago
Don Turner	f87e769b7d	Merge pull request #681 from SimonMarquis/di/coroutine-scope Add Application-wide `CoroutineScope` in the DI graph	1 year ago
Neelansh Sahai	bbdc62cbf4	Merge pull request #706 from gururani-abhishek/remove-onItemClick-parameter fix #614 : removed unused onItemClick parameter from NewsResourceCardList.kt	1 year ago
Alex Vanyo	76b0b08121	Adjust JDK setup to remove toolchain Change-Id: Ie0f43cb8f0fe504ed3e98ef35703949959688699	1 year ago
Milosz Moczkowski	e38e24eee9	Merge pull request #583 from SimonMarquis/jvmToolchain Configure jvmToolchain for Android and Kotlin projects	1 year ago
Adetunji Dahunsi	ba1d9e14df	Merge pull request #716 from SimonMarquis/SimonMarquis-patch-3 Simplify handling of nullability in `ConnectivityManagerNetworkMonitor`	1 year ago
Don Turner	a4eab289df	Merge pull request #719 from android/add-search-analytics Add analytics for search queries	1 year ago
James Rose	2f2dfb00a4	Mark all news resources read on first sync Otherwise, there will be unread articles until the user clicks through every article that has ever been published.	1 year ago
Don Turner	0e40e26f22	Add analytics for search queries Change-Id: If5290a903afb928fc1c3aaa66a52d2f182aba118	1 year ago
Simon Marquis	ec9155fef9	Update ConnectivityManagerNetworkMonitor.kt	2 years ago
Simon Marquis	7819a0c6bc	Extract common code to update the Flow value	2 years ago
Simon Marquis	def5c4e881	Update ConnectivityManagerNetworkMonitor.kt	2 years ago
Simon Marquis	76c249a651	Update ConnectivityManagerNetworkMonitor.kt	2 years ago
Simon Marquis	f34ee1c01b	Update ConnectivityManagerNetworkMonitor.kt	2 years ago
Simon Marquis	de6545f5d7	Simplify handling of nullability in `ConnectivityManagerNetworkMonitor`	2 years ago
TJ Dahunsi	6e6abd0326	Fix build	2 years ago
TJ Dahunsi	9562f7707d	Route notification deep link through for you screen, with spotless fixes	2 years ago
TJ Dahunsi	ef97cb941c	Route notification deep link through for you screen	2 years ago
TJ Dahunsi	8a3a16de21	Add pending intent for opening notification links	2 years ago
TJ Dahunsi	2499c0a0bd	Notify users when news are updated	2 years ago
Simon Marquis	ddabba8e4a	Merge branch 'main' into jvmToolchain	2 years ago
gururani-abhishek	f6bdba08d9	added comment to document what happens on tapping news resource card.	2 years ago
gururani-abhishek	bbc80bb0e2	added back launchCustomChromeTab()	2 years ago
gururani-abhishek	51379a249d	fix #614 : removed unused onItemClick parameter from NewsResourceCardList.kt 1. removed the unused onItemClick parameter 2. removed now redundant variables it used 3. removed its description from the comments.	2 years ago
Don Turner	2fd1b716f6	Merge pull request #667 from SimonMarquis/cleanup/warnings Cleanup warnings and replace deprecated methods	2 years ago
Milosz Moczkowski	c746b6afcd	Merge pull request #598 from takagimeow/fix/clean-up-unused-update-topics Clean up unused updateTopics and updateNewsResources methods	2 years ago
Simon Marquis	1341f7c2c5	Replace unused variable with simpler method calls	2 years ago
Simon Marquis	459717f8ba	Remove `DSL_SCOPE_VIOLATION` suppression as it is now fixed in Gradle 8.1	2 years ago
Don Turner	e1ae12129a	Merge pull request #662 from android/av/agp-8.0 Update to AGP 8.0	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
Simon Marquis	5a43b4b74e	Run Spotless	2 years ago
Simon Marquis	aa701064a2	Add Application-wide `CoroutineScope` in the DI graph Following the work of #607.	2 years ago
Simon Marquis	9499e559e7	Delete no longer used `AUTHORS_ASSET` reference	2 years ago
Milosz Moczkowski	6bf37adde9	Merge pull request #671 from android/kotlin-version-updates Update Kotlin, Compose compiler, serialization versions	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
Alex Vanyo	086dc957c0	Update to AGP 8.0 Change-Id: I27eb106e484e2003d732782a39612dc72c845282	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	0b0fdbfdbf	Merge branch 'main' into patch-2	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
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
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
Keisuke Takagi	d56d8e1380	Merge branch 'main' of https://github.com/takagimeow/nowinandroid into fix/clean-up-unused-update-topics	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
Adetunji Dahunsi	1e8d1d9810	Spotless fix Thought I was cool enough to edit code in the browser. Spotless said no.	2 years ago
Adetunji Dahunsi	267adfd27f	Batch sync news resources from remote Batch sync news resources from remote to reduce json payload size.	2 years ago
Alex Vanyo	0844d021b9	Merge pull request #603 from amalhanaja/redundant-managed-device Remove redundant code block for managed virtual device creation	2 years ago
Manuel Vivo	755413bfc5	Fix spotless	2 years ago

1 2 3 4 5 ...

314 Commits (374808c284172c3208ae7274658ee6effb930ec6)