Open CI links (#1902)

Author: rnystrom

Classes/Issues/Merge/IssueMergeContextModel.swift

@@ -17,21 +17,24 @@ final class IssueMergeContextModel: ListDiffable {
     let login: String
     let avatarURL: URL
     let description: String
+    let targetURL: URL
 
     init(
         id: String,
         context: String,
         state: StatusState,
         login: String,
         avatarURL: URL,
-        description: String
+        description: String,
+        targetURL: URL
         ) {
         self.id = id
         self.context = context
         self.state = state
         self.login = login
         self.avatarURL = avatarURL
         self.description = description
+        self.targetURL = targetURL
     }
 
     // MARK: ListDiffable
@@ -48,6 +51,7 @@ final class IssueMergeContextModel: ListDiffable {
         && login == object.login
         && avatarURL == object.avatarURL
         && description == object.description
+        && targetURL == object.targetURL
     }
 
 }

Classes/Issues/Merge/IssueMergeSectionController.swift

@@ -11,7 +11,8 @@ import IGListKit
 
 final class IssueMergeSectionController: ListBindingSectionController<IssueMergeModel>,
 ListBindingSectionControllerDataSource,
-MergeButtonDelegate {
+MergeButtonDelegate,
+ListBindingSectionControllerSelectionDelegate {
 
     private let model: IssueDetailsModel
     private let client: GithubClient
@@ -24,6 +25,7 @@ MergeButtonDelegate {
         self.resultID = resultID
         super.init()
         dataSource = self
+        selectionDelegate = self
     }
 
     // MARK: Private API
@@ -188,4 +190,15 @@ MergeButtonDelegate {
         viewController?.present(alert, animated: trueUnlessReduceMotionEnabled)
     }
 
+    // MARK: ListBindingSectionControllerSelectionDelegate
+
+    func sectionController(
+        _ sectionController: ListBindingSectionController<ListDiffable>,
+        didSelectItemAt index: Int,
+        viewModel: Any
+        ) {
+        guard let viewModel = viewModel as? IssueMergeContextModel else { return }
+        viewController?.presentSafari(url: viewModel.targetURL)
+    }
+
 }

Classes/Issues/PullRequest+IssueType.swift

@@ -59,15 +59,18 @@ extension IssueOrPullRequestQuery.Data.Repository.IssueOrPullRequest.AsPullReque
         var contexts = [IssueMergeContextModel]()
         for context in commit.status?.contexts ?? [] {
             guard let creator = context.creator,
-                let avatarURL = URL(string: creator.avatarUrl)
+                let avatarURL = URL(string: creator.avatarUrl),
+                let targetUrlString = context.targetUrl,
+                let targetURL = URL(string: targetUrlString)
                 else { continue }
             contexts.append(IssueMergeContextModel(
                 id: context.id,
                 context: context.context,
                 state: context.state,
                 login: creator.login,
                 avatarURL: avatarURL,
-                description: context.description ?? ""
+                description: context.description ?? "",
+                targetURL: targetURL
             ))
         }
 

gql/API.swift

@@ -12887,6 +12887,7 @@ public final class IssueOrPullRequestQuery: GraphQLQuery {
                       GraphQLField("state", type: .nonNull(.scalar(StatusState.self))),
                       GraphQLField("creator", type: .object(Creator.selections)),
                       GraphQLField("description", type: .scalar(String.self)),
+                      GraphQLField("targetUrl", type: .scalar(String.self)),
                     ]
 
                     public var snapshot: Snapshot
@@ -12895,8 +12896,8 @@ public final class IssueOrPullRequestQuery: GraphQLQuery {
                       self.snapshot = snapshot
                     }
 
-                    public init(id: GraphQLID, context: String, state: StatusState, creator: Creator? = nil, description: String? = nil) {
-                      self.init(snapshot: ["__typename": "StatusContext", "id": id, "context": context, "state": state, "creator": creator.flatMap { (value: Creator) -> Snapshot in value.snapshot }, "description": description])
+                    public init(id: GraphQLID, context: String, state: StatusState, creator: Creator? = nil, description: String? = nil, targetUrl: String? = nil) {
+                      self.init(snapshot: ["__typename": "StatusContext", "id": id, "context": context, "state": state, "creator": creator.flatMap { (value: Creator) -> Snapshot in value.snapshot }, "description": description, "targetUrl": targetUrl])
                     }
 
                     public var __typename: String {
@@ -12957,6 +12958,16 @@ public final class IssueOrPullRequestQuery: GraphQLQuery {
                       }
                     }
 
+                    /// The URL for this status context.
+                    public var targetUrl: String? {
+                      get {
+                        return snapshot["targetUrl"] as? String
+                      }
+                      set {
+                        snapshot.updateValue(newValue, forKey: "targetUrl")
+                      }
+                    }
+
                     public struct Creator: GraphQLSelectionSet {
                       public static let possibleTypes = ["Organization", "User", "Bot"]
 
@@ -18488,7 +18499,7 @@ public struct RepoEventFields: GraphQLFragment {
 
 public struct CommitContext: GraphQLFragment {
   public static let fragmentString =
-    "fragment commitContext on Commit {\n  __typename\n  id\n  status {\n    __typename\n    contexts {\n      __typename\n      id\n      context\n      state\n      creator {\n        __typename\n        login\n        avatarUrl\n      }\n      description\n    }\n    state\n  }\n}"
+    "fragment commitContext on Commit {\n  __typename\n  id\n  status {\n    __typename\n    contexts {\n      __typename\n      id\n      context\n      state\n      creator {\n        __typename\n        login\n        avatarUrl\n      }\n      description\n      targetUrl\n    }\n    state\n  }\n}"
 
   public static let possibleTypes = ["Commit"]
 
@@ -18594,6 +18605,7 @@ public struct CommitContext: GraphQLFragment {
         GraphQLField("state", type: .nonNull(.scalar(StatusState.self))),
         GraphQLField("creator", type: .object(Creator.selections)),
         GraphQLField("description", type: .scalar(String.self)),
+        GraphQLField("targetUrl", type: .scalar(String.self)),
       ]
 
       public var snapshot: Snapshot
@@ -18602,8 +18614,8 @@ public struct CommitContext: GraphQLFragment {
         self.snapshot = snapshot
       }
 
-      public init(id: GraphQLID, context: String, state: StatusState, creator: Creator? = nil, description: String? = nil) {
-        self.init(snapshot: ["__typename": "StatusContext", "id": id, "context": context, "state": state, "creator": creator.flatMap { (value: Creator) -> Snapshot in value.snapshot }, "description": description])
+      public init(id: GraphQLID, context: String, state: StatusState, creator: Creator? = nil, description: String? = nil, targetUrl: String? = nil) {
+        self.init(snapshot: ["__typename": "StatusContext", "id": id, "context": context, "state": state, "creator": creator.flatMap { (value: Creator) -> Snapshot in value.snapshot }, "description": description, "targetUrl": targetUrl])
       }
 
       public var __typename: String {
@@ -18664,6 +18676,16 @@ public struct CommitContext: GraphQLFragment {
         }
       }
 
+      /// The URL for this status context.
+      public var targetUrl: String? {
+        get {
+          return snapshot["targetUrl"] as? String
+        }
+        set {
+          snapshot.updateValue(newValue, forKey: "targetUrl")
+        }
+      }
+
       public struct Creator: GraphQLSelectionSet {
         public static let possibleTypes = ["Organization", "User", "Bot"]
 

gql/Fragments.graphql

@@ -104,6 +104,7 @@ fragment commitContext on Commit {
         avatarUrl
       }
       description
+      targetUrl
     }
     state
   }