nimblehq / jetpack-compose-crypto

This is an sample Crypto market prices app built with Jetpack Compose
https://nimblehq.co/
MIT License
21 stars 5 forks source link

[#3] [UI] As a user, I can switch between light/dark mode base on System settings #30

Closed luongvo closed 2 years ago

luongvo commented 2 years ago

Resolves https://github.com/nimblehq/jetpack-compose-crypto/issues/3

What happened 👀

Insight 📝

N/A

Proof Of Work 📹

https://user-images.githubusercontent.com/16315358/188708295-95fabc3a-2dea-4f1d-ae60-733e70b349b9.mp4

https://user-images.githubusercontent.com/16315358/188708333-35b2a664-ffe9-48cd-82ed-2e9a7caccc7d.mp4

github-actions[bot] commented 2 years ago
3 Warnings
:warning: app/src/main/AndroidManifest.xml#L10 - On SDK version 23 and up, your app data will be automatically backed up and restored on app install. Consider adding the attribute android:fullBackupContent to specify an @xml resource which configures which files to backup, or just set android:fullBackupOnly=true. More info: https://developer.android.com/guide/topics/data/autobackup
:warning: app/src/main/AndroidManifest.xml#L13 - Attribute networkSecurityConfig is only used in API level 24 and higher (current min is 23)
:warning: app/src/main/res/navigation/nav_graph_main.xml#L2 - The resource R.navigation.nav_graph_main appears to be unused

CoroutineTemplate Jacoco report:

Generated by :no_entry_sign: Danger

Wadeewee commented 2 years ago

@luongvo Could you please rebase branch with develop 🙏

luongvo commented 2 years ago

@luongvo Could you please rebase branch with develop 🙏

Done @Wadeewee