39 Commits (ca72d7d776b9486bd17e90d34f7ca90b6fe2d7db)

Author	SHA1	Message	Date
lihenggui	e03cf253b0	Unify style for di	1 year ago
lihenggui	a8aaeee69f	Inject network monitor and time monitor correctly	1 year ago
lihenggui	a928d2d724	Spotless	1 year ago
lihenggui	0e9d23318a	Use Koin for di in the common module	1 year ago
lihenggui	32918f929b	Merge remote-tracking branch 'upstream/main'	2 years ago
Simon Marquis	3dbca86a37	Merge remote-tracking branch 'origin/main' into jvm-alt	2 years ago
lihenggui	9420c0c770	Use @Dispatcher to distinguish different dispatchers	2 years ago
lihenggui	288ec388d7	Spotless	2 years ago
lihenggui	6982c57b93	Provide CoroutineScope and Dispatchers in common	2 years ago
lihenggui	678e98ee3b	Use correct implementation for providing dispatchers	2 years ago
lihenggui	b275cdcb79	Fix compilation errors in the ui module	2 years ago
lihenggui	234ad1e6cc	Change js to wasmJs	2 years ago
lihenggui	db8e99efdc	Add platform specific implementation for dispatchers	2 years ago
lihenggui	2f412db367	Use object to provide implementations	2 years ago
lihenggui	c5521ce89b	Make :core:common as a multiplatform library	2 years ago
Don Turner	522efac8c0	Update Compose BOM to 2024.02.00. Fixes #1218 ... Change-Id: I838b81bf8e7fd7cb135f422653a8d8041829b28b	2 years ago
Simon Marquis	e629d5e4bf	Merge branch 'main' into jvm-alt	2 years ago
Simon Marquis	61c7d71870	Migrate `:core:common` to pure JVM module ... The notification icons are now stored in `:core:notifications`. This forces `:sync:work` to depend on it. Another solution could be to provide the resource id through Hilt, but it would require more changes.	2 years ago
Don Turner	707117a4f2	Merge branch 'main' into kotlinify	2 years ago
Don Turner	0f0511748e	Merge pull request #1088 from SimonMarquis/cleanup-dependencies ... Reduce overall build complexity	2 years ago
Simon Marquis	f27a2e176e	Reduce overall build complexity ... - Reduce the visibility of multiple Hilt `Module`s and implementations of public interfaces - Correctly configure the visibility of dependencies: - `api` when it's part of it's public `api` - `implementation` when it's part of it's internal `implementation` - Remove unnecessary dependencies in build.gradle.kts files - Remove unnecessary dependencies provided by plugins - Remove unnecessary applied plugins (i.e. roborazzi) - Sort dependencies in `build.gradle.kts` - Delete unused entries in `libs.versions.toml`	2 years ago
Simon Marquis	caa482bc71	Kotlinify codebase ... - Remove unnecessary nullable types - Replace no-op method bodies with Unit - Convert to expression body - Replace if with when - Remove braces from 'when' entries - Remove braces from if statement - Convert to single line lambda - oneline if/returns - Replace 'contains' call with 'in' operator Following this refactor, it could be great to envision a more "strict" code formatter like ktlint 1.0 (we are currently stuck at 0.48.1)	2 years ago
Simon Marquis	b2cd980170	Merge remote-tracking branch 'origin/main' into resource-prefix	2 years ago
Simon Marquis	ebfe01affd	Enforce `resourcePrefix` on Android library modules ... ``` :ui-test-hilt-manifest -> ui_test_hilt_manifest_ :core:analytics -> core_analytics_ :core:common -> core_common_ :core:data -> core_data_ :core:data-test -> core_data_test_ :core:database -> core_database_ :core:datastore -> core_datastore_ :core:datastore-test -> core_datastore_test_ :core:designsystem -> core_designsystem_ :core:domain -> core_domain_ :core:network -> core_network_ :core:notifications -> core_notifications_ :core:testing -> core_testing_ :core:ui -> core_ui_ :feature:bookmarks -> feature_bookmarks_ :feature:foryou -> feature_foryou_ :feature:interests -> feature_interests_ :feature:search -> feature_search_ :feature:settings -> feature_settings_ :feature:topic -> feature_topic_ :sync:sync-test -> sync_test_ :sync:work -> sync_work_ ```	2 years ago
Simon Marquis	a33dafccad	Migrate from `java` to `kotlin` sourceSet directory	2 years ago
Don Turner	492697ce6b	Merge pull request #873 from SimonMarquis/misc-warnings ... Fix miscellaneous warnings	3 years ago
Don Turner	8901e5932a	Merge pull request #872 from SimonMarquis/data-objects ... Convert 'sealed' sub-object to 'data object'	3 years ago
Simon Marquis	e8449112b4	Replace StringDecoder injection with simpler API ... The initial reason for this `StringDecoder` was to make sure the AndroidX navigation route would not use incompatible chars in the resulting String. For that, `Uri.encode(String)` was used, but this is an Android API (`android.net.Uri`). Unit tests would fail because Uri is not mocked (since we are not using Robolectric). Therefore, a `StringDecoder` interface was introduced with 2 implementations: `UriDecoder` and `FakeStringDecoder`, Hilt modules were added, etc. FWIW, the naming of the API was misleading: `StringDecoder.decodeString(encodedString: String)` does not inform how encoded the input is, so `UriDecoder` could "decode" something that was not necessarily uri "encoded". The solution to this problem was to simply use regular Java APIs: - `URLDecoder.decode(urlEncodedTopicId, "UTF-8")` - `URLEncoder.encode(topicId, "UTF-8")`	3 years ago
Simon Marquis	4bdc9705af	Fix miscellaneous warnings ... - Remove unused namespace declaration - Collapse empty manifest tags - Remove redundant qualifier name - Remove redundant suppression	3 years ago
Simon Marquis	b7799f0475	Convert 'sealed' sub-object to 'data object'	3 years ago
Don Turner	4cdb5963a0	Add top level application scope, use IO dispatcher for DataStore ... Change-Id: I1512b1665587c73abd5e8a78aa9abd3eed24ab79	3 years ago
Simon Marquis	5a43b4b74e	Run Spotless	3 years ago
Simon Marquis	aa701064a2	Add Application-wide `CoroutineScope` in the DI graph ... Following the work of #607.	3 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.	3 years ago
Márton Braun	819dd494ad	Run Spotless to add trailing commas (and apply other rule updates) ... Change-Id: I5f8e49a3ef004c7e9e7dfe4bc55f0a35dc575209	3 years ago
Simon Marquis	90dc300783	Migrate from JUnit assertions to Kotlin's test framework ... Fixes #419	3 years ago
Manuel Vivo	a0b22d8ed2	Updates Navigation approach with new guidance (#347)	3 years ago
Wojtek Kaliciński	dcc23829d4	Migrate packages to namespaces in build files ... Change-Id: I21e021caa0ea72a7097dfd9af04a3c07a0cc9aa5	3 years ago
madroid	3bd81154b5	Move 'core-' module to core subfolder	4 years ago