forked from tangled.org/core
this repo has no description

appview: ingest pipeline records only for repos with spindles configured

also fixes a small UI bug with pipeline symbols on stacked PRs

Signed-off-by: oppiliappan <me@oppi.li>

oppi.li 684e939d b9f90a16

verified
Changed files
+17 -1
appview
pages
templates
repo
pulls
state
+1 -1
appview/pages/templates/repo/pulls/fragments/summarizedPullHeader.html
··· 17 17 {{ $latestRound := .LastRoundNumber }} 18 18 {{ $lastSubmission := index .Submissions $latestRound }} 19 19 {{ $commentCount := len $lastSubmission.Comments }} 20 - {{ if $pipeline }} 20 + {{ if and $pipeline $pipeline.Id }} 21 21 <div class="inline-flex items-center gap-2"> 22 22 {{ template "repo/pipelines/fragments/pipelineSymbol" $pipeline }} 23 23 <span class="mx-2 before:content-['·'] before:select-none"></span>
+16
appview/state/knotstream.go
··· 140 140 return fmt.Errorf("empty repo: nsid %s, rkey %s", msg.Nsid, msg.Rkey) 141 141 } 142 142 143 + // does this repo have a spindle configured? 144 + repos, err := db.GetRepos( 145 + d, 146 + db.FilterEq("did", record.TriggerMetadata.Repo.Did), 147 + db.FilterEq("name", record.TriggerMetadata.Repo.Repo), 148 + ) 149 + if err != nil { 150 + return fmt.Errorf("failed to look for repo in DB: nsid %s, rkey %s, %w", msg.Nsid, msg.Rkey, err) 151 + } 152 + if len(repos) != 1 { 153 + return fmt.Errorf("incorrect number of repos returned: %d (expected 1)", len(repos)) 154 + } 155 + if repos[0].Spindle == "" { 156 + return fmt.Errorf("repo does not have a spindle configured yet: nsid %s, rkey %s", msg.Nsid, msg.Rkey) 157 + } 158 + 143 159 // trigger info 144 160 var trigger db.Trigger 145 161 var sha string