Commit Graph

465 Commits (709732834a837d924124eadd57d7f12e073609a8)

Author SHA1 Message Date
Simon Marquis 0ced769999 Widen `isNotEmpty` with `isNotBlank` to potentially catch blank values
1 year ago
Simon Marquis 4a5ec5bd82 Update `var` with fake default value to the more "correct" `lateinit var`
1 year ago
Simon Marquis f1bc6f1226 Restore `Unknown` type as empty `String`
1 year ago
Simon Marquis c831dc673c
Fix broken text
1 year ago
Simon Marquis daa2228132 Restore original `type` labels
1 year ago
Simon Marquis 480d8e7d7c Fix missing removal
1 year ago
Simon Marquis 553b01fcb6 spotlessApply
1 year ago
Simon Marquis 0377fd8712 Replace `NewsResourceType` enum with simpler String type
1 year ago
Adetunji Dahunsi 153b34fd50
Merge pull request #850 from qamarelsafadi/loading-progress-for-image
1 year ago
RivanParmar 9e1d46e28c Rename SearchResult.kt.kt to SearchResult.kt
1 year ago
Don Turner 492697ce6b
Merge pull request #873 from SimonMarquis/misc-warnings
1 year ago
qamarelsafadi 308f51f87d Merge remote-tracking branch 'origin/loading-progress-for-image' into loading-progress-for-image
1 year ago
Qamar A. Safadi 2db3e0c804
Merge branch 'android:main' into loading-progress-for-image
1 year ago
qamarelsafadi 5d7e33d905 fix formatting
1 year ago
Don Turner 8901e5932a
Merge pull request #872 from SimonMarquis/data-objects
1 year ago
Don Turner 0e13207a40
Merge pull request #874 from SimonMarquis/string-decoder
1 year ago
qamarelsafadi f203e9a218 feat: replacing SubcomposeAsyncImage as its not recommended to use with lazyLayout
1 year ago
Qamar A. Safadi 5abc69fb49
Merge branch 'android:main' into loading-progress-for-image
1 year ago
Don Turner 4d0efef3b4
Merge pull request #879 from tatsuyafujisaki/rename-constants
1 year ago
AlirezaJoon a23ffab602 fix: add label to animations for fixing lint issue
1 year ago
Qamar A. Safadi 8aa0457ca9
Merge branch 'android:main' into loading-progress-for-image
1 year ago
Tatsuya Fujisaki 4eeb045287 Rename constants
1 year ago
Adetunji Dahunsi ec53d03854
Merge pull request #722 from android/tj/scrollbars
1 year ago
Simon Marquis e8449112b4 Replace StringDecoder injection with simpler API
1 year ago
Simon Marquis 4bdc9705af Fix miscellaneous warnings
1 year ago
Simon Marquis b7799f0475 Convert 'sealed' sub-object to 'data object'
1 year ago
qamarelsafadi 5bbcc904e9 feat: make painter in one line
1 year ago
qamarelsafadi 3595d8dcb5 feat: move the placeholder default value to the method
1 year ago
Qamar A. Safadi 9f3103c212
Merge branch 'android:main' into loading-progress-for-image
1 year ago
prosixe c779b4e8ca Typo fix for topics
1 year ago
qamarelsafadi e276212574 feat: remove check if placeholder null and put a default placeholder if its not passed.
1 year ago
qamarelsafadi 9c1ec17a39 fix: modifier to Modifier
1 year ago
qamarelsafadi 57025e7587 fix: review notes
1 year ago
qamarelsafadi 8a197800c5 feat: fix format issue
1 year ago
qamarelsafadi f4859e0bcd Merge remote-tracking branch 'origin/loading-progress-for-image' into loading-progress-for-image
1 year ago
Qamar A. Safadi 06e91fd623
Merge branch 'android:main' into loading-progress-for-image
1 year ago
qamarelsafadi a294adb6b9 feat: fix format issue
1 year ago
Adetunji Dahunsi 48041fcaa4
Merge pull request #835 from blackbracken/avoid_retrieving_unnecessary_entity
1 year ago
qamarelsafadi 7a05f7db96 feat: Show loading progress for Image Component as a solve for
1 year ago
qamarelsafadi 3284eb68a3 feat: Show loading progress for Image Component as a solve for
1 year ago
qamarelsafadi 42b16bf7be Merge remote-tracking branch 'origin/main'
1 year ago
Qamar Safadi ee8b70003d feat: Show loading progress for Image Component as a solve for
1 year ago
TJ Dahunsi 9babb50c58 Interact with scrollbar only if drag direction matches scrollbar orientation
1 year ago
TJ Dahunsi e13e84c155 Replace usages of 'displacement' to 'moved' in parameter names
1 year ago
TJ Dahunsi 806726a7f0 PR feedback
1 year ago
TJ Dahunsi eeb4973265 PR feedback
1 year ago
TJ Dahunsi 2dba522663 Check if first item size == 0
1 year ago
TJ Dahunsi cb1d50e65e Even better names
1 year ago
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
1 year ago
Simon Marquis f7ed38182d Add protobuf generated sources to the sourceSets
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
1 year ago
Don Turner b2a74ba174
Merge pull request #812 from Askeri-Muhendis/main
2 years ago
TJ Dahunsi c4686cfa54 Track when deep link notifications are opened
2 years ago
ahmedmadhoun1 3114b2eeaa Allow localised date format
2 years ago
Simon Marquis d136965a0d Merge branch 'main' into lint/sarif
2 years ago
İbrahim Ethem Şen 6271d1c4c0
Merge branch 'android:main' into main
2 years ago
İbrahim Ethem Şen 56f8a27132 Kotlin coding conventions places companion object at the bottom of the class
2 years ago
Tomáš Mlynarič 8e1e994bb1 Add named arguments
2 years ago
Tomáš Mlynarič 4a8a43551a Add contentType to news feed
2 years ago
Simon Marquis 8e708cca05
Merge branch 'main' into lint/sarif
2 years ago
James Rose 7826b4f5f2
Merge pull request #748 from SimonMarquis/fix/permissions
2 years ago
Adetunji Dahunsi 890c0f5769
Merge pull request #776 from SimonMarquis/fix/714
2 years ago
Simon Marquis 27154ef38a Apply changes from review comments
2 years ago
Miłosz Moczkowski 479a2f00b0 Refactor icons
2 years ago
Milosz Moczkowski 1345764649
Merge pull request #770 from android/refactor/remove_unused_statements
2 years ago
Simon Marquis fecab96e4c Keep track of matching `Network`s inside `NetworkCallback`
2 years ago
Miłosz Moczkowski 99094199bc Remove redundant suppressions
2 years ago
Miłosz Moczkowski 5bf66739bd Remove redundant statements
2 years ago
Simon Marquis 9aba5cc0fe Cleanup unused string resources
2 years ago
Simon Marquis 8ea9a8fa51 Android Lint improvements
2 years ago
Adetunji Dahunsi f914254414
Merge pull request #749 from android/tj/reactive-only-dao
2 years ago
Don Turner d319264ef0 Provide Default dispatcher for instrumented tests
2 years ago
Don Turner b245334a18 Updating TestDataStoreModule to use ApplicationScope
2 years ago
TJ Dahunsi dcc36b0228 Fix tests
2 years ago
Don Turner 4cdb5963a0 Add top level application scope, use IO dispatcher for DataStore
2 years ago
TJ Dahunsi cd6ad7d575 Ensure DAO exposes only reactive types
2 years ago
Simon Marquis 823c4db201 Grant `POST_NOTIFICATIONS` permission in more instrumented tests
2 years ago
Don Turner f87e769b7d
Merge pull request #681 from SimonMarquis/di/coroutine-scope
2 years ago
Neelansh Sahai bbdc62cbf4
Merge pull request #706 from gururani-abhishek/remove-onItemClick-parameter
2 years ago
Alex Vanyo 76b0b08121 Adjust JDK setup to remove toolchain
2 years ago
Milosz Moczkowski e38e24eee9
Merge pull request #583 from SimonMarquis/jvmToolchain
2 years ago
Adetunji Dahunsi ba1d9e14df
Merge pull request #716 from SimonMarquis/SimonMarquis-patch-3
2 years ago
Don Turner a4eab289df
Merge pull request #719 from android/add-search-analytics
2 years ago
James Rose 2f2dfb00a4 Mark all news resources read on first sync
2 years ago
Don Turner 0e40e26f22 Add analytics for search queries
2 years 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
2 years ago
Don Turner 2fd1b716f6
Merge pull request #667 from SimonMarquis/cleanup/warnings
2 years ago
Milosz Moczkowski c746b6afcd
Merge pull request #598 from takagimeow/fix/clean-up-unused-update-topics
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
2 years ago
Takeshi Hagikura b3cdc172cd
Implement search feature (#685)
2 years ago
Adetunji Dahunsi 73a38720d8
Merge pull request #649 from android/tj/backend-requested-sync
2 years ago
Adetunji Dahunsi 6b834b6f4c Applied code review suggestions
2 years ago
Adetunji Dahunsi b3f2502ec2
Update core/data/src/test/java/com/google/samples/apps/nowinandroid/core/data/repository/OfflineFirstNewsRepositoryTest.kt
2 years ago
Adetunji Dahunsi f86d174c1b Used more explicit variable names in tests
2 years ago
Adetunji Dahunsi 337c940d66 Add comments explaining backend sync and made variable names easier to grok
2 years ago
Adetunji Dahunsi 022cd92f4c
Update core/data/src/main/java/com/google/samples/apps/nowinandroid/core/data/repository/OfflineFirstNewsRepository.kt
2 years ago
Simon Marquis 5a43b4b74e Run Spotless
2 years ago
Simon Marquis aa701064a2 Add Application-wide `CoroutineScope` in the DI graph
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
2 years ago
Márton Braun bea2eecd90 Update Kotlin, Compose compiler, serialization versions
2 years ago
Adetunji Dahunsi 05be2855d8 Wire up backend requested sync
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
2 years ago
Milosz Moczkowski feafb5fbed
Merge pull request #575 from takagimeow/fix/provide-true-to-local-inspection-mode
2 years ago
Milosz Moczkowski 2b2f6d48ce
Merge pull request #600 from takagimeow/cleanup-unused-args
2 years ago
Milosz Moczkowski 0b0fdbfdbf
Merge branch 'main' into patch-2
2 years ago
Adetunji Dahunsi 08956492c8 Backend triggered sync
2 years ago
Milosz Moczkowski ba6a697844
Merge pull request #624 from SimonMarquis/gradle-sort
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
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
2 years ago
James Rose bd450099fb Replace GetUserNewsResourcesUseCase with UserNewsResourceRepository
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
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
2 years ago
Adetunji Dahunsi 8765d3ac57 Fix tests for batched news resource sync
2 years ago
Adetunji Dahunsi 1e8d1d9810
Spotless fix
2 years ago
Adetunji Dahunsi 267adfd27f
Batch sync news resources from remote
2 years ago
Alex Vanyo 0844d021b9
Merge pull request #603 from amalhanaja/redundant-managed-device
2 years ago
Manuel Vivo 755413bfc5 Fix spotless
2 years ago
Manuel Vivo 91633af376 Provide a CoroutineScope to fake DataStores
2 years ago
Don Turner 2f3fc85d65 Make Hilt provide Firebase.analytics
2 years ago
AlfianHanantio bcd4a6de59 Remove redundant code block for managed virtual device creation
2 years ago
Keisuke Takagi 072364e7f1 remove newsResourceTitle
2 years ago
Keisuke Takagi 0a748bc899 Merge branch 'main' of github.com:takagimeow/nowinandroid into fix/clean-up-unused-update-topics
2 years ago
Adetunji Dahunsi ae58d1e1b0
Merge pull request #592 from android/tj/news-resource-query
2 years ago
Adetunji Dahunsi 09f5c3bc61 Add NewsResourceQuery to better query encapsulation
2 years ago
Keisuke Takagi 86c021a54a remove invalid overriding method
2 years ago
Keisuke Takagi fb32580e0c remove unused updateNewsResources method
2 years ago
Keisuke Takagi 67412b2720 remove invalid overriding method
2 years ago
Keisuke Takagi 1489bf66e3 remove unused updateTopics method
2 years ago
Caren 6a5feb767e
Merge pull request #582 from yschimke/loaders
2 years ago
Keisuke Takagi 8b51107d7a Merge branch 'main' of github.com:takagimeow/nowinandroid into fix/provide-true-to-local-inspection-mode
2 years ago
Simon Marquis e1f8c51c05 Configure jvmToolchain on the single Kotlin jvm project
2 years ago
Yuri Schimke 4fa730098d Fixes
2 years ago
Yuri Schimke 9e399b69ea Improve network pooling and avoid startup image fetches.
2 years ago
Don Turner 4284e6912e Merge remote-tracking branch 'github/main' into feb9automerger
2 years ago
Márton Braun 237375f52a Remove unused DropdownMenu component
2 years ago
Márton Braun 322ff63dbc Route topic chip tap events up to NiaNavHost
2 years ago
Don Turner 2a4d0434a5 Add Firebase analytics. See go/nia-firebase-services.
2 years ago
Keisuke Takagi 9091abf09e Provide true to LocalInspectionMode
2 years ago
Simon Marquis 3409f937ed Migrate custom room configuration into a convention plugin
2 years ago
Miłosz Moczkowski bcb4d7d592 Replace global variables with PreviewParameterProvider
2 years ago
Márton Braun 819dd494ad Run Spotless to add trailing commas (and apply other rule updates)
2 years ago
Angie Sasmita cfbf782613
Merge pull request #542 from android/as/dynamic-color-option
2 years ago
mlykotom 6b48b355cc Merge branch 'main' into tm/fix-benchmarks-no-people
2 years ago
Mohsen Rzna 46c1d510d4
Android-CI workflow -> GMD refactor (#537)
2 years ago
Angie Sasmita dbbe8ffecb Small fix for naming and boolean logic
2 years ago
Angie Sasmita 19efc1b5e8 Fix Spotless issue manually
2 years ago
Angie Sasmita 6046f6e943 Add dynamic color option in settings for API >=32
2 years ago
Takeshi Hagikura 4633609930
Introduce GMD + ATD (#479)
2 years ago
Don Turner bad3ea60f5 Refactor Topics news feed, tidy a few other bits up
2 years ago
magicalmeghan 45e3cfd989 fixed string issue for testTopicsChipColorBackground_matchesFollowedState test
2 years ago
magicalmeghan de6a4ec6c7 Merge remote-tracking branch 'github/mm/chipcolor' into mm/chipcolor
2 years ago
magicalmeghan 5137cd7777 removed hardcoded strings
2 years ago
Don Turner cafac8eb09 fix spotless, tidy test data
2 years ago
magicalmeghan 66046a86b0 Updated NewsResourceCardTest
2 years ago
Don Turner 6207fe3bcf mapToUserNewsResources converts List<NewsResource> to List<UserNewsResource>
2 years ago
Don Turner 24df51349d Move mapping logic into secondary constructor
2 years ago
Don Turner 355b0540aa Move construction of UserNewsResource into separate function
2 years ago
Don Turner 99227b06b5 Spotless
2 years ago
Don Turner 32bc5c1590 Fix merge conflicts
2 years ago
magicalmeghan ffb5aeba86 Added third draft of tests
2 years ago
magicalmeghan d9e18607be Added second draft of tests
2 years ago
magicalmeghan 8089866079 Added second draft of tests
2 years ago
Alex Vanyo ba11e6bf38
Merge pull request #491 from android/av/fix-background-composable
2 years ago
Chris Sinco 2f454de21c Spacing polish based on design spec
2 years ago
Don Turner 47bc4a7b88
Merge pull request #504 from android/dt/fix-build-warnings
2 years ago
Don Turner 0917cdebec Refactor NiaGradientBackground to accept GradientColors
2 years ago