Add specific UX for Expected UTDs due to membership

element-hq · Apr 26, 2024 · d7dcca6 · d7dcca6
1 parent df6cf25
commit d7dcca6
Show file tree

Hide file tree

Showing 5 changed files with 60 additions and 11 deletions.
diff --git a/ElementX.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved b/ElementX.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
@@ -121,8 +121,8 @@
       "kind" : "remoteSourceControl",
       "location" : "https://github.com/matrix-org/matrix-analytics-events",
       "state" : {
-        "revision" : "f756bf0756b7349a1d3ccee0d038790d1ab2ec56",
-        "version" : "0.14.0"
+        "revision" : "e4e49896331c9dcf7346c90529a9aad7944a3259",
+        "version" : "0.20.0"
       }
     },
     {

diff --git a/ElementX/Sources/FlowCoordinators/UserSessionFlowCoordinator.swift b/ElementX/Sources/FlowCoordinators/UserSessionFlowCoordinator.swift
@@ -156,8 +156,13 @@ class UserSessionFlowCoordinator: FlowCoordinatorProtocol {
                 } else {
                     timeToDecryptMs = -1
                 }
-
-                analytics.trackError(context: nil, domain: .E2EE, name: .OlmKeysNotSentError, timeToDecryptMillis: timeToDecryptMs)
+
+                switch info.cause {
+                case .unknown:
+                    analytics.trackError(context: nil, domain: .E2EE, name: .OlmKeysNotSentError, timeToDecryptMillis: timeToDecryptMs)
+                case .membership:
+                    analytics.trackError(context: nil, domain: .E2EE, name: .HistoricalMessage, timeToDecryptMillis: timeToDecryptMs)
+                }
             }
             .store(in: &cancellables)
     }

diff --git a/ElementX/Sources/Screens/RoomScreen/View/Timeline/EncryptedRoomTimelineView.swift b/ElementX/Sources/Screens/RoomScreen/View/Timeline/EncryptedRoomTimelineView.swift
@@ -14,14 +14,29 @@
 // limitations under the License.
 //
 
+import Compound
 import SwiftUI
 
 struct EncryptedRoomTimelineView: View {
     let timelineItem: EncryptedRoomTimelineItem
 
+    var icon: KeyPath<CompoundIcons, Image> {
+        switch timelineItem.encryptionType {
+        case .megolmV1AesSha2(_, let cause):
+            switch cause {
+            case .unknown:
+                return \.time
+            case .membership:
+                return \.block
+            }
+        default:
+            return \.time
+        }
+    }
+
     var body: some View {
         TimelineStyler(timelineItem: timelineItem) {
-            Label(timelineItem.body, icon: \.time, iconSize: .small, relativeTo: .compound.bodyLG)
+            Label(timelineItem.body, icon: icon, iconSize: .small, relativeTo: .compound.bodyLG)
                 .labelStyle(RoomTimelineViewLabelStyle())
                 .font(.compound.bodyLG)
         }
@@ -68,6 +83,10 @@ struct EncryptedRoomTimelineView_Previews: PreviewProvider, TestablePreview {
                                                              timestamp: "Later",
                                                              isOutgoing: true,
                                                              senderId: "Anne"))
+
+            EncryptedRoomTimelineView(timelineItem: expectedItemWith(timestamp: "Now",
+                                                                     isOutgoing: false,
+                                                                     senderId: "Bob"))
         }
     }
 
@@ -81,4 +100,15 @@ struct EncryptedRoomTimelineView_Previews: PreviewProvider, TestablePreview {
                                   canBeRepliedTo: false,
                                   sender: .init(id: senderId))
     }
+
+    private static func expectedItemWith(timestamp: String, isOutgoing: Bool, senderId: String) -> EncryptedRoomTimelineItem {
+        EncryptedRoomTimelineItem(id: .random,
+                                  body: "You don't have access to this message",
+                                  encryptionType: .megolmV1AesSha2(sessionID: "foo", cause: .membership),
+                                  timestamp: timestamp,
+                                  isOutgoing: isOutgoing,
+                                  isEditable: false,
+                                  canBeRepliedTo: false,
+                                  sender: .init(id: senderId))
+    }
 }
diff --git a/ElementX/Sources/Services/Analytics/AnalyticsService.swift b/ElementX/Sources/Services/Analytics/AnalyticsService.swift
@@ -139,7 +139,17 @@ extension AnalyticsService {
     /// Can be found in `UnableToDecryptInfo`. In case the `UnableToDecryptInfo` contains the value as nil, pass it as `-1`
     func trackError(context: String?, domain: AnalyticsEvent.Error.Domain, name: AnalyticsEvent.Error.Name, timeToDecryptMillis: Int? = nil) {
         // CryptoModule is deprecated
-        capture(event: AnalyticsEvent.Error(context: context, cryptoModule: nil, cryptoSDK: .Rust, domain: domain, name: name, timeToDecryptMillis: timeToDecryptMillis))
+        capture(event: AnalyticsEvent.Error(context: context,
+                                            cryptoModule: .Rust,
+                                            cryptoSDK: .Rust,
+                                            domain: domain,
+                                            eventLocalAgeMillis: nil,
+                                            isFederated: nil,
+                                            isMatrixDotOrg: nil,
+                                            name: name,
+                                            timeToDecryptMillis: timeToDecryptMillis,
+                                            userTrustsOwnIdentity: nil,
+                                            wasVisibleToUser: nil))
     }
 
     /// Track the creation of a room

diff --git a/ElementX/Sources/Services/Timeline/TimelineItems/RoomTimelineItemFactory.swift b/ElementX/Sources/Services/Timeline/TimelineItems/RoomTimelineItemFactory.swift
@@ -161,21 +161,25 @@ struct RoomTimelineItemFactory: RoomTimelineItemFactoryProtocol {
                                             _ encryptedMessage: EncryptedMessage,
                                             _ isOutgoing: Bool) -> RoomTimelineItemProtocol {
         var encryptionType = EncryptedRoomTimelineItem.EncryptionType.unknown
+        var errorLabel = L10n.commonWaitingForDecryptionKey
         switch encryptedMessage {
         case .megolmV1AesSha2(let sessionID, let cause):
-            let cause: EncryptedRoomTimelineItem.UTDCause = switch cause {
-            case .unknown: .unknown
-            case .membership: .membership
+            switch cause {
+            case .unknown:
+                encryptionType = .megolmV1AesSha2(sessionID: sessionID, cause: .unknown)
+                errorLabel = L10n.commonWaitingForDecryptionKey
+            case .membership:
+                encryptionType = .megolmV1AesSha2(sessionID: sessionID, cause: .membership)
+                errorLabel = L10n.commonUnableToDecryptNoAccess
             }
-            encryptionType = .megolmV1AesSha2(sessionID: sessionID, cause: cause)
         case .olmV1Curve25519AesSha2(let senderKey):
             encryptionType = .olmV1Curve25519AesSha2(senderKey: senderKey)
         case .unknown:
             break
         }
 
         return EncryptedRoomTimelineItem(id: eventItemProxy.id,
-                                         body: L10n.commonWaitingForDecryptionKey,
+                                         body: errorLabel,
                                          encryptionType: encryptionType,
                                          timestamp: eventItemProxy.timestamp.formatted(date: .omitted, time: .shortened),
                                          isOutgoing: isOutgoing,