appview/{pipelines,pages}: filter pipelines list by push/PR

Signed-off-by: Anirudh Oppiliappan <[e-mail]>

appview/pages/pages.go

···
 }
 
 type PipelinesParams struct {
-	LoggedInUser *oauth.MultiAccountUser
-	RepoInfo     repoinfo.RepoInfo
-	Pipelines    []models.Pipeline
-	Active       string
+	LoggedInUser    *oauth.MultiAccountUser
+	RepoInfo        repoinfo.RepoInfo
+	Pipelines       []models.Pipeline
+	Active          string
+	FilteringByPush bool
+	FilteringByPR   bool
 }
 
 func (p *Pages) Pipelines(w io.Writer, params PipelinesParams) error {

appview/pages/templates/repo/pipelines/pipelines.html

···
 {{ end }}
 
 {{ define "repoContent" }}
-<div class="flex justify-between items-center gap-4 mb-4">
-  <div class="text-sm text-gray-600 dark:text-gray-400">
-    {{ len .Pipelines }} pipeline run{{ if ne (len .Pipelines) 1 }}s{{ end }}
+  {{ $active := "all" }}
+  {{ if .FilteringByPush }}
+    {{ $active = "push" }}
+  {{ else if .FilteringByPR }}
+    {{ $active = "pr" }}
+  {{ end }}
+
+  {{ $all :=
+     (dict
+       "Key" "all"
+       "Value" "all"
+       "Icon" "package"
+       "Meta" "") }}
+  {{ $push :=
+     (dict
+       "Key" "push"
+       "Value" "push"
+       "Icon" "git-commit-horizontal"
+       "Meta" "") }}
+  {{ $pr :=
+     (dict
+       "Key" "pr"
+       "Value" "pull request"
+       "Icon" "git-pull-request"
+       "Meta" "") }}
+  {{ $values := list $all $push $pr }}
+
+  <div class="flex justify-between items-center gap-4">
+    <div>
+      {{ template "fragments/tabSelector" (dict "Name" "trigger" "Values" $values "Active" $active) }}
+    </div>
+    <div class="text-sm text-gray-600 dark:text-gray-400">
+      {{ len .Pipelines }} pipeline run{{ if ne (len .Pipelines) 1 }}s{{ end }}
+    </div>
   </div>
-  <!-- Future: Add filter tabs here similar to issues/pulls -->
-</div>
 {{ end }}
 
 {{ define "repoAfter" }}

appview/pipelines/pipelines.go

···
 		return
 	}
 
+	// Filter by trigger
+	filterTrigger := r.URL.Query().Get("trigger")
+	var filtered []models.Pipeline
+	for _, pipeline := range ps {
+		if filterTrigger == "push" && pipeline.Trigger != nil && pipeline.Trigger.IsPush() {
+			filtered = append(filtered, pipeline)
+		} else if filterTrigger == "pr" && pipeline.Trigger != nil && pipeline.Trigger.IsPullRequest() {
+			filtered = append(filtered, pipeline)
+		} else if filterTrigger == "" || filterTrigger == "all" {
+			filtered = append(filtered, pipeline)
+		}
+	}
+
+	filteringByPush := filterTrigger == "push"
+	filteringByPR := filterTrigger == "pr"
+
 	p.pages.Pipelines(w, pages.PipelinesParams{
-		LoggedInUser: user,
-		RepoInfo:     p.repoResolver.GetRepoInfo(r, user),
-		Pipelines:    ps,
+		LoggedInUser:    user,
+		RepoInfo:        p.repoResolver.GetRepoInfo(r, user),
+		Pipelines:       filtered,
+		FilteringByPush: filteringByPush,
+		FilteringByPR:   filteringByPR,
 	})
 }