Conversation
- aims to have a centralised place for all feature login, overrideable by forks and debug flavours
- adds a dedicated routing activity to proxy the uri to the login selected by the feature flags
| } else { | ||
| clickOn(R.string.settings_notification_advanced) | ||
| pressBack() | ||
| when (BuildConfig.NOTIFICATION_SETTINGS_VERSION!!) { |
There was a problem hiding this comment.
Why not using a DefaultVectorFeatures instance here? It would prevent the usage of !! (I think)
There was a problem hiding this comment.
good point! I wasn't sure how the injections would work in the SanityTests, I would invest more time but this is removed in #4627
| refreshBackgroundSyncPrefs() | ||
|
|
||
| handleSystemPreference() | ||
| handleVersionedSettings() |
There was a problem hiding this comment.
Can we observe some blink with this new way to hide preference? Maybe hide both pref by default in the XML
There was a problem hiding this comment.
there's a blink if the visibility is updated in onResume but onBindPref is instant (as long as we stay on the main thread)
the programmatic visibility is also removed in #4627 🎉
| } | ||
| } | ||
|
|
||
| private fun handleVersionedSettings() { |
There was a problem hiding this comment.
this change is temporary as V2 is live, V1 is removed in #4627
2 participants
Separates the SSO redirection from the
LoginActivity+LoginActivity2by using a dedicated invisibleSSORedirectRouterActivity, this allows both activities to be enabled at the same time and not exportedIntroduces a
VectorFeaturesabstraction to start grouping our feature flags, this will also enable overriding at runtime and provides an entry point for forks to extend to help avoid conflicts (debug overrides to come in the next PR)Ports the
LoginVersionandNotificationSettingsVersionto theVectorFeatures