Merge pull request #1312 from vector-im/feature/post_smoke_test_fix

Feature/post smoke test fix

vector/src/main/java/im/vector/riotx/features/home/HomeDetailFragment.kt

@@ -113,7 +113,7 @@ class HomeDetailFragment @Inject constructor(
                 VerificationVectorAlert(
                         uid = uid,
                         title = getString(R.string.new_session),
-                        description = getString(R.string.new_session_review_with_info, newest.displayName ?: "", newest.deviceId ?: ""),
+                        description = getString(R.string.verify_this_session, newest.displayName ?: newest.deviceId ?: ""),
                         iconId = R.drawable.ic_shield_warning
                 ).apply {
                     matrixItem = user

vector/src/main/java/im/vector/riotx/features/roommemberprofile/devices/DeviceListEpoxyController.kt

@@ -63,7 +63,7 @@ class DeviceListEpoxyController @Inject constructor(private val stringProvider:
                 }
             }
             is Success    -> {
-                val deviceList = data.cryptoDevices.invoke().sortedByDescending {
+                val deviceList = data.cryptoDevices.invoke().sortedBy {
                     it.isVerified
                 }
 

vector/src/main/res/values/strings.xml

@@ -2354,6 +2354,8 @@ Not all features in Riot are implemented in RiotX yet. Main missing (and coming
     <string name="encrypted_unverified">Encrypted by an unverified device</string>
     <string name="review_logins">Review where you’re logged in</string>
     <string name="verify_other_sessions">Verify all your sessions to ensure your account &amp; messages are safe</string>
+    <!-- Argument will be replaced by the other session name (e.g, Riot Desktop, mobile) -->
+    <string name="verify_this_session">Verify the new login accessing your account: %1$s</string>
 
     <string name="cross_signing_verify_by_text">Manually Verify by Text</string>
     <string name="crosssigning_verify_session">Verify login</string>