Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MOB-363: Android: "System" theme setting not working #32

Merged
merged 2 commits into from
Mar 18, 2024

Conversation

ruixhuang
Copy link
Contributor

@ruixhuang ruixhuang commented Mar 15, 2024

Observe the system-side theme setting.

Screen_Recording_20240315_184624_dYdX.Debug.mp4

Copy link

linear bot commented Mar 15, 2024

@ruixhuang ruixhuang marked this pull request as draft March 15, 2024 22:18
Copy link
Contributor

@prashanDYDX prashanDYDX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm actually surprised this fixes the issue while the app is running - since it seems like createFromPreference() should only be called once at app start, right?

@prashanDYDX
Copy link
Contributor

Also, may be easier to handle this at the Compose level via isSystemInDarkTheme()

@ruixhuang ruixhuang marked this pull request as ready for review March 15, 2024 22:52
@ruixhuang
Copy link
Contributor Author

I'm actually surprised this fixes the issue while the app is running - since it seems like createFromPreference() should only be called once at app start, right?

Also when user changes theme from settings

@ruixhuang
Copy link
Contributor Author

Also, may be easier to handle this at the Compose level via isSystemInDarkTheme()

I didn't know this one :) But it's a Compose function, so we can't use it here.

@ruixhuang ruixhuang requested a review from prashanDYDX March 15, 2024 22:54
@prashanDYDX
Copy link
Contributor

Also, may be easier to handle this at the Compose level via isSystemInDarkTheme()

I didn't know this one :) But it's a Compose function, so we can't use it here.

Right yeah, you'd do it at the root Compose container view level.

Comment on lines +167 to +168
// This is a state that is used to force a recomposition when the theme changes.
private var themeChangedState by mutableIntStateOf(0)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We shouldn't need to track this ourselves - isSystemInDarkTheme() handles this for us. It is backed by a CompositionLocal of the Configuration:

@Composable
@ReadOnlyComposable
internal actual fun _isSystemInDarkTheme(): Boolean {
    val uiMode = LocalConfiguration.current.uiMode
    return (uiMode and Configuration.UI_MODE_NIGHT_MASK) == Configuration.UI_MODE_NIGHT_YES
}

Copy link
Contributor

@prashanDYDX prashanDYDX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Discussed offline - going to land this as is to unblock the release. In the medium term we should try and re-architect this to use the native Compose functions and features (replace shared instance with CompositionLocal).

@ruixhuang ruixhuang merged commit 885b278 into develop Mar 18, 2024
3 checks passed
@ruixhuang ruixhuang deleted the feature/system_theme branch March 18, 2024 22:31
mike-dydx pushed a commit that referenced this pull request Apr 11, 2024
* Update .gitignore to ignore all .idea files and all build directories. (#20)

* Turn on Gradle parallel execution and caching.

* Add "app_deployment" key (#21)

* Add "app_deployment" key

* Lint

* Initial setup of StrictMode - only logging in debug. (#22)

* Initial setup of StrictMode - only logging in debug.

* Revert injection changes

* Use limitedParallelism and MainScope() in AbacusThreadingImp. (#25)

* Handle exception with invalid URLs (#26)

* MOB-348 Show Squid routing error on deposit/withdrawal (#28)

* Update AppModule.kt and used dependencies to follow Dagger Best Practices guide (in Notion). (#27)

* Correlate x-request-id to correlate route requests from Squid transactions (#23)

* Fix build (#29)

* MOB-362: Some url links don't work with the deployment build (#31)

* Test

* Check for deeplink path during routing

* Clean up

* Clean up

* Update rest of Dagger modules to follow best practices. (#30)

* Remove network selector from settings for release build (#33)

* MOB-363: Android: "System" theme setting not working (#32)

* Observe system theme

* Recompose when theme changes

* Fix a crash on url tapping (#34)

* Remove hardcoded scheme in the setting links. (#35)

* Add Firebase Performance monitoring. (#37)

* Bump Abacus and update the system link. (#36)

* Fix the release build issue of wallet amount not showing up on deposit (#38)

* Fix Subaccount transfer for France (#39)

* Fix Subaccount transfer for France

* Lint

* Update function name to be consistent with iOS

* Clean up the start-up sequence (#41)

* Clean up the start-up sequence

* Lint

* Update Cartera and WalletConnnect to the latest version (#40)

* Update Cartera

* Lint

* Add comments

* MOB-285: Call startTrade() after setMarket() (#43)

* Update the v4-native-client.js to replace # with ___ (#42)

* Add script

* Update v4-client.js

* Safer replacement

* Safer replacement

* Move startWorkers back to Activity.onCreate() (#44)

* Move startWorkers back to Activity.start()

* Comment

* MOB-410 Add tracking for onboarding, transfer, wallet connection (#46)

* Adding Onboarding/Transfer/Wallet events

* Add userID and userProperties

* Optimiazation

* Add link to send logcat messages via email (#45)

* Add link to send logcat messages via email

* Clean up

* Error handling

* Make file provider depending on applicationId

* MOB-432 previous branch was based on a wrong branch (#48)

* MOB-432 previous branch was based on a wrong branch

* PR

* Bump Abacus (#50)

* Bump Abacus

* Revert

* MOB-421 add settings control to turn in-app notifications on/off completely  (#49)

* MOB-421 add settings control to turn in-app notifications on/off completely

* MOB-421 add settings control to turn in-app notifications on/off completely

* Clean up

* Feature/mob 396 position cell with margin type (#51)

* MOB-432 previous branch was based on a wrong branch

* PR

* MOB-396 skeleton for position cell

* Better layout for place holder views (when there is no position, fill or transfers)

* Convert AbacusState flows into StateFlows. (#52)

* MOB-422 Withdrawal gating (#53)

* Withdrawal Gating

* Clean up

* Doing the paddings better. (#56)

* Update Android Gradle Plugin (#55)

* MOB-446  create new add price triggers panel screen  (#58)

* SL/TP Routing and  price triggers panel screen

* Update deeplink to match iOS

* Renaming

* MOB-443 add limit price to price triggers screen (#60)

* Feature/mob 356 trade input (#57)

* Margin type and leverage screens skeleton

* MOB-356 MOB-358 Margin mode screen

* Change bg color

* move modifier to param

* Feature/mob 257 weekly dates (#61)

* Added end time to rewards date range

* lint

* Update SharingStarted usages from Lazily to WhileSubscribed(). (#62)

* Fix issue of text input losing calculated value when not focused (#64)

* Inject coroutine scopes and dispatchers instead of hardcoding. (#65)

* Revert "Update SharingStarted usages from Lazily to WhileSubscribed(). (#62)" (#66)

This reverts commit de98092.

* Fix margin issues with portfolio position items (#67)

* Fix threading issue (#68)

* Bump Abacus and v4-client (#69)

---------

Co-authored-by: prashanDYDX <163016611+prashanDYDX@users.noreply.github.com>
Co-authored-by: Prashan Dharmasena <prashan@dydx.exchange>
Co-authored-by: John Huang <johnqh@yahoo.com>
ruixhuang added a commit that referenced this pull request Apr 30, 2024
* Update .gitignore to ignore all .idea files and all build directories. (#20)

* Turn on Gradle parallel execution and caching.

* Add "app_deployment" key (#21)

* Add "app_deployment" key

* Lint

* Initial setup of StrictMode - only logging in debug. (#22)

* Initial setup of StrictMode - only logging in debug.

* Revert injection changes

* Use limitedParallelism and MainScope() in AbacusThreadingImp. (#25)

* Handle exception with invalid URLs (#26)

* MOB-348 Show Squid routing error on deposit/withdrawal (#28)

* Update AppModule.kt and used dependencies to follow Dagger Best Practices guide (in Notion). (#27)

* Correlate x-request-id to correlate route requests from Squid transactions (#23)

* Fix build (#29)

* MOB-362: Some url links don't work with the deployment build (#31)

* Test

* Check for deeplink path during routing

* Clean up

* Clean up

* Update rest of Dagger modules to follow best practices. (#30)

* Remove network selector from settings for release build (#33)

* MOB-363: Android: "System" theme setting not working (#32)

* Observe system theme

* Recompose when theme changes

* Fix a crash on url tapping (#34)

* Remove hardcoded scheme in the setting links. (#35)

* Add Firebase Performance monitoring. (#37)

* Bump Abacus and update the system link. (#36)

* Fix the release build issue of wallet amount not showing up on deposit (#38)

* Fix Subaccount transfer for France (#39)

* Fix Subaccount transfer for France

* Lint

* Update function name to be consistent with iOS

* Clean up the start-up sequence (#41)

* Clean up the start-up sequence

* Lint

* Update Cartera and WalletConnnect to the latest version (#40)

* Update Cartera

* Lint

* Add comments

* MOB-285: Call startTrade() after setMarket() (#43)

* Update the v4-native-client.js to replace # with ___ (#42)

* Add script

* Update v4-client.js

* Safer replacement

* Safer replacement

* Move startWorkers back to Activity.onCreate() (#44)

* Move startWorkers back to Activity.start()

* Comment

* MOB-410 Add tracking for onboarding, transfer, wallet connection (#46)

* Adding Onboarding/Transfer/Wallet events

* Add userID and userProperties

* Optimiazation

* Add link to send logcat messages via email (#45)

* Add link to send logcat messages via email

* Clean up

* Error handling

* Make file provider depending on applicationId

* MOB-432 previous branch was based on a wrong branch (#48)

* MOB-432 previous branch was based on a wrong branch

* PR

* Bump Abacus (#50)

* Bump Abacus

* Revert

* MOB-421 add settings control to turn in-app notifications on/off completely  (#49)

* MOB-421 add settings control to turn in-app notifications on/off completely

* MOB-421 add settings control to turn in-app notifications on/off completely

* Clean up

* Feature/mob 396 position cell with margin type (#51)

* MOB-432 previous branch was based on a wrong branch

* PR

* MOB-396 skeleton for position cell

* Better layout for place holder views (when there is no position, fill or transfers)

* Convert AbacusState flows into StateFlows. (#52)

* MOB-422 Withdrawal gating (#53)

* Withdrawal Gating

* Clean up

* Doing the paddings better. (#56)

* Update Android Gradle Plugin (#55)

* MOB-446  create new add price triggers panel screen  (#58)

* SL/TP Routing and  price triggers panel screen

* Update deeplink to match iOS

* Renaming

* MOB-443 add limit price to price triggers screen (#60)

* Feature/mob 356 trade input (#57)

* Margin type and leverage screens skeleton

* MOB-356 MOB-358 Margin mode screen

* Change bg color

* move modifier to param

* Feature/mob 257 weekly dates (#61)

* Added end time to rewards date range

* lint

* Update SharingStarted usages from Lazily to WhileSubscribed(). (#62)

* Fix issue of text input losing calculated value when not focused (#64)

* Inject coroutine scopes and dispatchers instead of hardcoding. (#65)

* Revert "Update SharingStarted usages from Lazily to WhileSubscribed(). (#62)" (#66)

This reverts commit de98092.

* Fix margin issues with portfolio position items (#67)

* Fix threading issue (#68)

* Bump Abacus and v4-client (#69)

* MOB-440 TP/SL order submission and data binding (#63)

* Wire up input data

* Submission

* WIP

* Bump Abacus and toggle size section

* Update PlatformInfoScaffold

* submissionStatus = _submissionStatus

* Simplify TriggerOrderStream

* Fix an issue of order/fill status display (#72)

* Fix dollar() formatting crash in FR locale. (#74)

* Add formatting functions for size specified in double (1000.0, 0.001,… (#73)

* Add formatting functions for size specified in double (1000.0, 0.001, etc)

* Clean up

* Clean up

* Clean up BigDecimals

* Remove rounded(bigDecimal:

* Fix rounding issue on leverage slider when locale is French (#75)

* MOB-441 add TP/SL display to market screen (Android) (#71)

* Wire up input data

* Submission

* WIP

* Bump Abacus and toggle size section

* MarketInfo buttons

* Clean up

* Clean up TriggerOrderStream

* MOB-442 Support error states for TP/SL inputs  (#77)

* Wire up input data

* Submission

* WIP

* Bump Abacus and toggle size section

* MarketInfo buttons

* Validation

* Show validation error at sections

* Highlighting error/warning field

* Slide size formatting

* Clean up

* Clean up TriggerOrderStream

* Clean up

* Clean up

* Made localizer a computed property (#78)

* MOB-455 Add retry to trade and close order submission (#76)

* Add retry to trade and close order submission.

* _submissionStatus.asStateFlow()

* Turn on Kotlin incremental classpath snapshots. (#80)

* Fixing threading issues (#79)

* Fixing threading issues.

* Clean up

* Feature/mob 360 target leverage (#59)

* Margin type and leverage screens skeleton

* MOB-356 MOB-358 Margin mode screen

* Change bg color

* MOB-360 rough UX for target leverage screen

* Fixed PR

* move modifier to param

* lint

* There is no longer InputFieldScarfold

* Put back InputFieldScaffold

* Fix market info's stats/about tab animation issue (#83)

* MOB-14 Disable sparkline touching on market list (#84)

* Disable sparkline touching on market list

* Clean up

* Bump version (#81)

* Bump Abacus and add script to use locally built Abacus (#86)

* Bump Abacus and add script to use locally built Abacus

* Usage

* Update

* Feature/mob 400 adjust margin screen (#85)

* Margin type and leverage screens skeleton

* MOB-356 MOB-358 Margin mode screen

* Change bg color

* MOB-360 rough UX for target leverage screen

* Fixed PR

* move modifier to param

* In the middle of coding

* lint

* There is no longer InputFieldScarfold

* Put back InputFieldScaffold

* More placeholder code and it compiles

* rough amount input

* Formatting "Add Margin" and "Remove Margin"

---------

Co-authored-by: prashanDYDX <163016611+prashanDYDX@users.noreply.github.com>
Co-authored-by: Prashan Dharmasena <prashan@dydx.exchange>
Co-authored-by: John Huang <johnqh@yahoo.com>
johnqh added a commit that referenced this pull request May 21, 2024
* Update .gitignore to ignore all .idea files and all build directories. (#20)

* Turn on Gradle parallel execution and caching.

* Add "app_deployment" key (#21)

* Add "app_deployment" key

* Lint

* Initial setup of StrictMode - only logging in debug. (#22)

* Initial setup of StrictMode - only logging in debug.

* Revert injection changes

* Use limitedParallelism and MainScope() in AbacusThreadingImp. (#25)

* Handle exception with invalid URLs (#26)

* MOB-348 Show Squid routing error on deposit/withdrawal (#28)

* Update AppModule.kt and used dependencies to follow Dagger Best Practices guide (in Notion). (#27)

* Correlate x-request-id to correlate route requests from Squid transactions (#23)

* Fix build (#29)

* MOB-362: Some url links don't work with the deployment build (#31)

* Test

* Check for deeplink path during routing

* Clean up

* Clean up

* Update rest of Dagger modules to follow best practices. (#30)

* Remove network selector from settings for release build (#33)

* MOB-363: Android: "System" theme setting not working (#32)

* Observe system theme

* Recompose when theme changes

* Fix a crash on url tapping (#34)

* Remove hardcoded scheme in the setting links. (#35)

* Add Firebase Performance monitoring. (#37)

* Bump Abacus and update the system link. (#36)

* Fix the release build issue of wallet amount not showing up on deposit (#38)

* Fix Subaccount transfer for France (#39)

* Fix Subaccount transfer for France

* Lint

* Update function name to be consistent with iOS

* Clean up the start-up sequence (#41)

* Clean up the start-up sequence

* Lint

* Update Cartera and WalletConnnect to the latest version (#40)

* Update Cartera

* Lint

* Add comments

* MOB-285: Call startTrade() after setMarket() (#43)

* Update the v4-native-client.js to replace # with ___ (#42)

* Add script

* Update v4-client.js

* Safer replacement

* Safer replacement

* Move startWorkers back to Activity.onCreate() (#44)

* Move startWorkers back to Activity.start()

* Comment

* MOB-410 Add tracking for onboarding, transfer, wallet connection (#46)

* Adding Onboarding/Transfer/Wallet events

* Add userID and userProperties

* Optimiazation

* Add link to send logcat messages via email (#45)

* Add link to send logcat messages via email

* Clean up

* Error handling

* Make file provider depending on applicationId

* MOB-432 previous branch was based on a wrong branch (#48)

* MOB-432 previous branch was based on a wrong branch

* PR

* Bump Abacus (#50)

* Bump Abacus

* Revert

* MOB-421 add settings control to turn in-app notifications on/off completely  (#49)

* MOB-421 add settings control to turn in-app notifications on/off completely

* MOB-421 add settings control to turn in-app notifications on/off completely

* Clean up

* Feature/mob 396 position cell with margin type (#51)

* MOB-432 previous branch was based on a wrong branch

* PR

* MOB-396 skeleton for position cell

* Better layout for place holder views (when there is no position, fill or transfers)

* Convert AbacusState flows into StateFlows. (#52)

* MOB-422 Withdrawal gating (#53)

* Withdrawal Gating

* Clean up

* Doing the paddings better. (#56)

* Update Android Gradle Plugin (#55)

* MOB-446  create new add price triggers panel screen  (#58)

* SL/TP Routing and  price triggers panel screen

* Update deeplink to match iOS

* Renaming

* MOB-443 add limit price to price triggers screen (#60)

* Feature/mob 356 trade input (#57)

* Margin type and leverage screens skeleton

* MOB-356 MOB-358 Margin mode screen

* Change bg color

* move modifier to param

* Feature/mob 257 weekly dates (#61)

* Added end time to rewards date range

* lint

* Update SharingStarted usages from Lazily to WhileSubscribed(). (#62)

* Fix issue of text input losing calculated value when not focused (#64)

* Inject coroutine scopes and dispatchers instead of hardcoding. (#65)

* Revert "Update SharingStarted usages from Lazily to WhileSubscribed(). (#62)" (#66)

This reverts commit de98092.

* Fix margin issues with portfolio position items (#67)

* Fix threading issue (#68)

* Bump Abacus and v4-client (#69)

* MOB-440 TP/SL order submission and data binding (#63)

* Wire up input data

* Submission

* WIP

* Bump Abacus and toggle size section

* Update PlatformInfoScaffold

* submissionStatus = _submissionStatus

* Simplify TriggerOrderStream

* Fix an issue of order/fill status display (#72)

* Fix dollar() formatting crash in FR locale. (#74)

* Add formatting functions for size specified in double (1000.0, 0.001,… (#73)

* Add formatting functions for size specified in double (1000.0, 0.001, etc)

* Clean up

* Clean up

* Clean up BigDecimals

* Remove rounded(bigDecimal:

* Fix rounding issue on leverage slider when locale is French (#75)

* MOB-441 add TP/SL display to market screen (Android) (#71)

* Wire up input data

* Submission

* WIP

* Bump Abacus and toggle size section

* MarketInfo buttons

* Clean up

* Clean up TriggerOrderStream

* MOB-442 Support error states for TP/SL inputs  (#77)

* Wire up input data

* Submission

* WIP

* Bump Abacus and toggle size section

* MarketInfo buttons

* Validation

* Show validation error at sections

* Highlighting error/warning field

* Slide size formatting

* Clean up

* Clean up TriggerOrderStream

* Clean up

* Clean up

* Made localizer a computed property (#78)

* MOB-455 Add retry to trade and close order submission (#76)

* Add retry to trade and close order submission.

* _submissionStatus.asStateFlow()

* Turn on Kotlin incremental classpath snapshots. (#80)

* Fixing threading issues (#79)

* Fixing threading issues.

* Clean up

* Feature/mob 360 target leverage (#59)

* Margin type and leverage screens skeleton

* MOB-356 MOB-358 Margin mode screen

* Change bg color

* MOB-360 rough UX for target leverage screen

* Fixed PR

* move modifier to param

* lint

* There is no longer InputFieldScarfold

* Put back InputFieldScaffold

* Fix market info's stats/about tab animation issue (#83)

* MOB-14 Disable sparkline touching on market list (#84)

* Disable sparkline touching on market list

* Clean up

* Bump version (#81)

* Bump Abacus and add script to use locally built Abacus (#86)

* Bump Abacus and add script to use locally built Abacus

* Usage

* Update

* Feature/mob 400 adjust margin screen (#85)

* Margin type and leverage screens skeleton

* MOB-356 MOB-358 Margin mode screen

* Change bg color

* MOB-360 rough UX for target leverage screen

* Fixed PR

* move modifier to param

* In the middle of coding

* lint

* There is no longer InputFieldScarfold

* Put back InputFieldScaffold

* More placeholder code and it compiles

* rough amount input

* Formatting "Add Margin" and "Remove Margin"

* MOB-466 Integrate Abacus changes for trigger order status notification (#89)

* MOB-466 Integrate Abacus changes for trigger order status notification

* Clean up PlatformInfoScaffold

* Update FF logic to SL/TP trigger (#91)

* MOB-445 Add TP | SL button to market info page when there are open positions for the market  (#82)

* MOB-469: Add basic order lines to market chart (#90)

* MOB-469: Add basic order lines to market chart

* Only display open orders.

* MOB-468: Clean up market prices chart (#92)

* Clean up SL/TP (#93)

* Clean up SL/TP

* Cleanup

* MOB-470 MOB-471 Add order line annotations. (#94)

* Add Portfolio header selector (#95)

* Features/mob 400 adjust margin screen bottom (#87)

* Margin type and leverage screens skeleton

* MOB-356 MOB-358 Margin mode screen

* Change bg color

* MOB-360 rough UX for target leverage screen

* Fixed PR

* move modifier to param

* In the middle of coding

* lint

* There is no longer InputFieldScarfold

* Put back InputFieldScaffold

* More placeholder code and it compiles

* rough amount input

* Formatting "Add Margin" and "Remove Margin"

* MOB-400 placeholder for bottom

* Adjust margin receipt area

* MOB-400 rearranging receipt data

* MOB-400 liquidation price

* Fix copy and modified code

* PR review

* spotless

* fix build

* Update abacus to 1.7.6 (#97)

* MOB-480 MOB-484: Order-lines formatting, localization, rendering clean-up. (#96)

* Update Abacus (#98)

* Bump v4-client and AGP (#101)

* MOB-495: Add entry and liquidation lines. (#99)

* Feature/mob 497 integration margin type (#102)

* Set margin type into tradeInput

* Layout changes

* Update Abacus

* Fine tuning Isolated margin trade input

* Margin dialog fine tuning

* MOB-487 Android: Log error into Crashlytics (#100)

* WIP

* Timber

* Use Logging with consistency

* Cleanup

* Clean up

* Clean up

* Lint

* Feature/mob 498 integration target leverage (#103)

* UX working

* Sync up text formatting

* Update Abacus

* compile issue

* Feature/fix trigger (#104)

* Turn on V2 abacus manager in debug. (#106)

* Bump Cartera and add log message (#108)

* Reduct params of v4-client calls from logs (#107)

* Stop app from exiting at region restriction (#110)

* Stop app from exiting at region restriction

* Not showing the button

* Lint

* MOB-511 Fix JS runner in webview of older devices (#109)

* Fix deployment issues with mainnet test users (#111)

* Stop app from exiting at region restriction

* Not showing the button

* Lint

* Force TESTNET when user debugging is enabled.

* Force deployment web host

* Fix URL

* Clean up

* Made ethereumAddress null instead empty string

* Update version to 1.0.2

* Increase heap size

* Update from TESTNET to TESTFLIGHT

* Fix build

---------

Co-authored-by: prashanDYDX <163016611+prashanDYDX@users.noreply.github.com>
Co-authored-by: Prashan Dharmasena <prashan@dydx.exchange>
Co-authored-by: John Huang <johnqh@yahoo.com>
ruixhuang added a commit that referenced this pull request May 23, 2024
* Update .gitignore to ignore all .idea files and all build directories. (#20)

* Turn on Gradle parallel execution and caching.

* Add "app_deployment" key (#21)

* Add "app_deployment" key

* Lint

* Initial setup of StrictMode - only logging in debug. (#22)

* Initial setup of StrictMode - only logging in debug.

* Revert injection changes

* Use limitedParallelism and MainScope() in AbacusThreadingImp. (#25)

* Handle exception with invalid URLs (#26)

* MOB-348 Show Squid routing error on deposit/withdrawal (#28)

* Update AppModule.kt and used dependencies to follow Dagger Best Practices guide (in Notion). (#27)

* Correlate x-request-id to correlate route requests from Squid transactions (#23)

* Fix build (#29)

* MOB-362: Some url links don't work with the deployment build (#31)

* Test

* Check for deeplink path during routing

* Clean up

* Clean up

* Update rest of Dagger modules to follow best practices. (#30)

* Remove network selector from settings for release build (#33)

* MOB-363: Android: "System" theme setting not working (#32)

* Observe system theme

* Recompose when theme changes

* Fix a crash on url tapping (#34)

* Remove hardcoded scheme in the setting links. (#35)

* Add Firebase Performance monitoring. (#37)

* Bump Abacus and update the system link. (#36)

* Fix the release build issue of wallet amount not showing up on deposit (#38)

* Fix Subaccount transfer for France (#39)

* Fix Subaccount transfer for France

* Lint

* Update function name to be consistent with iOS

* Clean up the start-up sequence (#41)

* Clean up the start-up sequence

* Lint

* Update Cartera and WalletConnnect to the latest version (#40)

* Update Cartera

* Lint

* Add comments

* MOB-285: Call startTrade() after setMarket() (#43)

* Update the v4-native-client.js to replace # with ___ (#42)

* Add script

* Update v4-client.js

* Safer replacement

* Safer replacement

* Move startWorkers back to Activity.onCreate() (#44)

* Move startWorkers back to Activity.start()

* Comment

* MOB-410 Add tracking for onboarding, transfer, wallet connection (#46)

* Adding Onboarding/Transfer/Wallet events

* Add userID and userProperties

* Optimiazation

* Add link to send logcat messages via email (#45)

* Add link to send logcat messages via email

* Clean up

* Error handling

* Make file provider depending on applicationId

* MOB-432 previous branch was based on a wrong branch (#48)

* MOB-432 previous branch was based on a wrong branch

* PR

* Bump Abacus (#50)

* Bump Abacus

* Revert

* MOB-421 add settings control to turn in-app notifications on/off completely  (#49)

* MOB-421 add settings control to turn in-app notifications on/off completely

* MOB-421 add settings control to turn in-app notifications on/off completely

* Clean up

* Feature/mob 396 position cell with margin type (#51)

* MOB-432 previous branch was based on a wrong branch

* PR

* MOB-396 skeleton for position cell

* Better layout for place holder views (when there is no position, fill or transfers)

* Convert AbacusState flows into StateFlows. (#52)

* MOB-422 Withdrawal gating (#53)

* Withdrawal Gating

* Clean up

* Doing the paddings better. (#56)

* Update Android Gradle Plugin (#55)

* MOB-446  create new add price triggers panel screen  (#58)

* SL/TP Routing and  price triggers panel screen

* Update deeplink to match iOS

* Renaming

* MOB-443 add limit price to price triggers screen (#60)

* Feature/mob 356 trade input (#57)

* Margin type and leverage screens skeleton

* MOB-356 MOB-358 Margin mode screen

* Change bg color

* move modifier to param

* Feature/mob 257 weekly dates (#61)

* Added end time to rewards date range

* lint

* Update SharingStarted usages from Lazily to WhileSubscribed(). (#62)

* Fix issue of text input losing calculated value when not focused (#64)

* Inject coroutine scopes and dispatchers instead of hardcoding. (#65)

* Revert "Update SharingStarted usages from Lazily to WhileSubscribed(). (#62)" (#66)

This reverts commit de98092.

* Fix margin issues with portfolio position items (#67)

* Fix threading issue (#68)

* Bump Abacus and v4-client (#69)

* MOB-440 TP/SL order submission and data binding (#63)

* Wire up input data

* Submission

* WIP

* Bump Abacus and toggle size section

* Update PlatformInfoScaffold

* submissionStatus = _submissionStatus

* Simplify TriggerOrderStream

* Fix an issue of order/fill status display (#72)

* Fix dollar() formatting crash in FR locale. (#74)

* Add formatting functions for size specified in double (1000.0, 0.001,… (#73)

* Add formatting functions for size specified in double (1000.0, 0.001, etc)

* Clean up

* Clean up

* Clean up BigDecimals

* Remove rounded(bigDecimal:

* Fix rounding issue on leverage slider when locale is French (#75)

* MOB-441 add TP/SL display to market screen (Android) (#71)

* Wire up input data

* Submission

* WIP

* Bump Abacus and toggle size section

* MarketInfo buttons

* Clean up

* Clean up TriggerOrderStream

* MOB-442 Support error states for TP/SL inputs  (#77)

* Wire up input data

* Submission

* WIP

* Bump Abacus and toggle size section

* MarketInfo buttons

* Validation

* Show validation error at sections

* Highlighting error/warning field

* Slide size formatting

* Clean up

* Clean up TriggerOrderStream

* Clean up

* Clean up

* Made localizer a computed property (#78)

* MOB-455 Add retry to trade and close order submission (#76)

* Add retry to trade and close order submission.

* _submissionStatus.asStateFlow()

* Turn on Kotlin incremental classpath snapshots. (#80)

* Fixing threading issues (#79)

* Fixing threading issues.

* Clean up

* Feature/mob 360 target leverage (#59)

* Margin type and leverage screens skeleton

* MOB-356 MOB-358 Margin mode screen

* Change bg color

* MOB-360 rough UX for target leverage screen

* Fixed PR

* move modifier to param

* lint

* There is no longer InputFieldScarfold

* Put back InputFieldScaffold

* Fix market info's stats/about tab animation issue (#83)

* MOB-14 Disable sparkline touching on market list (#84)

* Disable sparkline touching on market list

* Clean up

* Bump version (#81)

* Bump Abacus and add script to use locally built Abacus (#86)

* Bump Abacus and add script to use locally built Abacus

* Usage

* Update

* Feature/mob 400 adjust margin screen (#85)

* Margin type and leverage screens skeleton

* MOB-356 MOB-358 Margin mode screen

* Change bg color

* MOB-360 rough UX for target leverage screen

* Fixed PR

* move modifier to param

* In the middle of coding

* lint

* There is no longer InputFieldScarfold

* Put back InputFieldScaffold

* More placeholder code and it compiles

* rough amount input

* Formatting "Add Margin" and "Remove Margin"

* MOB-466 Integrate Abacus changes for trigger order status notification (#89)

* MOB-466 Integrate Abacus changes for trigger order status notification

* Clean up PlatformInfoScaffold

* Update FF logic to SL/TP trigger (#91)

* MOB-445 Add TP | SL button to market info page when there are open positions for the market  (#82)

* MOB-469: Add basic order lines to market chart (#90)

* MOB-469: Add basic order lines to market chart

* Only display open orders.

* MOB-468: Clean up market prices chart (#92)

* Clean up SL/TP (#93)

* Clean up SL/TP

* Cleanup

* MOB-470 MOB-471 Add order line annotations. (#94)

* Add Portfolio header selector (#95)

* Features/mob 400 adjust margin screen bottom (#87)

* Margin type and leverage screens skeleton

* MOB-356 MOB-358 Margin mode screen

* Change bg color

* MOB-360 rough UX for target leverage screen

* Fixed PR

* move modifier to param

* In the middle of coding

* lint

* There is no longer InputFieldScarfold

* Put back InputFieldScaffold

* More placeholder code and it compiles

* rough amount input

* Formatting "Add Margin" and "Remove Margin"

* MOB-400 placeholder for bottom

* Adjust margin receipt area

* MOB-400 rearranging receipt data

* MOB-400 liquidation price

* Fix copy and modified code

* PR review

* spotless

* fix build

* Update abacus to 1.7.6 (#97)

* MOB-480 MOB-484: Order-lines formatting, localization, rendering clean-up. (#96)

* Update Abacus (#98)

* Bump v4-client and AGP (#101)

* MOB-495: Add entry and liquidation lines. (#99)

* Feature/mob 497 integration margin type (#102)

* Set margin type into tradeInput

* Layout changes

* Update Abacus

* Fine tuning Isolated margin trade input

* Margin dialog fine tuning

* MOB-487 Android: Log error into Crashlytics (#100)

* WIP

* Timber

* Use Logging with consistency

* Cleanup

* Clean up

* Clean up

* Lint

* Feature/mob 498 integration target leverage (#103)

* UX working

* Sync up text formatting

* Update Abacus

* compile issue

* Feature/fix trigger (#104)

* Turn on V2 abacus manager in debug. (#106)

* Bump Cartera and add log message (#108)

* Reduct params of v4-client calls from logs (#107)

* Stop app from exiting at region restriction (#110)

* Stop app from exiting at region restriction

* Not showing the button

* Lint

* MOB-511 Fix JS runner in webview of older devices (#109)

* Fix deployment issues with mainnet test users (#111)

* Stop app from exiting at region restriction

* Not showing the button

* Lint

* Force TESTNET when user debugging is enabled.

* Force deployment web host

* Fix URL

* Clean up

* Made ethereumAddress null instead empty string

* Update version to 1.0.2

* Increase heap size

* Update from TESTNET to TESTFLIGHT

* MOB-513 Clean up force update logic (#113)

* MOB-516 Add NavigatePage event (#114)

* Bump app version to 1.0.3 (#115)

---------

Co-authored-by: prashanDYDX <163016611+prashanDYDX@users.noreply.github.com>
Co-authored-by: Prashan Dharmasena <prashan@dydx.exchange>
Co-authored-by: John Huang <johnqh@yahoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants