17 Commits (151f877bbe6546c78f8f03f16e56564e285fb508)

Author	SHA1	Message	Date
Don Turner	492697ce6b	Merge pull request #873 from SimonMarquis/misc-warnings ... Fix miscellaneous warnings	1 year ago
Don Turner	8901e5932a	Merge pull request #872 from SimonMarquis/data-objects ... Convert 'sealed' sub-object to 'data object'	1 year 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")`	1 year ago
Simon Marquis	4bdc9705af	Fix miscellaneous warnings ... - Remove unused namespace declaration - Collapse empty manifest tags - Remove redundant qualifier name - Remove redundant suppression	1 year ago
Simon Marquis	b7799f0475	Convert 'sealed' sub-object to 'data object'	1 year ago
Don Turner	4cdb5963a0	Add top level application scope, use IO dispatcher for DataStore ... Change-Id: I1512b1665587c73abd5e8a78aa9abd3eed24ab79	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
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
Márton Braun	819dd494ad	Run Spotless to add trailing commas (and apply other rule updates) ... Change-Id: I5f8e49a3ef004c7e9e7dfe4bc55f0a35dc575209	2 years ago
Simon Marquis	90dc300783	Migrate from JUnit assertions to Kotlin's test framework ... Fixes #419	2 years ago
Manuel Vivo	a0b22d8ed2	Updates Navigation approach with new guidance (#347)	2 years ago
Wojtek Kaliciński	dcc23829d4	Migrate packages to namespaces in build files ... Change-Id: I21e021caa0ea72a7097dfd9af04a3c07a0cc9aa5	2 years ago
Milosz Moczkowski	06e60063b5	Add dependency graphs to every module ... Change-Id: I629fef818f7521c13bb3069a3ff9ed9bc42e57f3	2 years ago
Wojtek Kaliciński	4683918cc7	Move spotless to init script ... Also adds automatic git hooks installation Change-Id: I18debbee43af27db7b95a4202f824fa87e186713	2 years ago
Wojtek Kaliciński	d0cff2b11c	Build cleanup ... * Move Hilt setup to convention plugin * Disable Hilt on modules where it's not needed * Enable non-transitive R classes * Fix `configuration` blocks called in wrong scope * Move imperative Flavors code from build.gradle * Disable unnecessary build features globally * Disable configure-on-demand as it's not encouraged * Don't use internal AGP packages in plugins Change-Id: I1bee3e3fd0103054637b979a350f9fd2312ae8ba	2 years ago
madroid	3bd81154b5	Move 'core-' module to core subfolder	2 years ago