Monorepo for Tangled tangled.org

appview/gfi: render gfi label when there isn't any matching issues #702

merged opened by boltless.me targeting master from sandboxed-atmosphere

although it might not happen, the appview should be able to handle when there is no single issue with good-first-issue label

Signed-off-by: Seongmin Lee git@boltless.me

Labels

None yet.

assignee

None yet.

Participants 2
AT URI
at://did:plc:xasnlahkri4ewmbuzly2rlc5/sh.tangled.repo.pull/3m3sa6t262h22
+1 -1
Interdiff #2 #3
+1 -1
appview/state/gfi.go
··· 28 gfiLabelDef, err := db.GetLabelDefinition(s.db, db.FilterEq("at_uri", goodFirstIssueLabel)) 29 if err != nil { 30 log.Println("failed to get gfi label def", err) 31 - s.pages.Error503(w) 32 return 33 } 34
··· 28 gfiLabelDef, err := db.GetLabelDefinition(s.db, db.FilterEq("at_uri", goodFirstIssueLabel)) 29 if err != nil { 30 log.Println("failed to get gfi label def", err) 31 + s.pages.Error500(w) 32 return 33 } 34

History

8 rounds 2 comments
sign up or login to add to the discussion
1 commit
expand
appview/gfi: render gfi label when there isn't any matching issues
expand 0 comments
pull request successfully merged
1 commit
expand
appview/gfi: render gfi label when there isn't any matching issues
expand 0 comments
1 commit
expand
appview/gfi: render gfi label when there isn't any matching issues
expand 0 comments
1 commit
expand
appview/gfi: render gfi label when there isn't any matching issues
expand 0 comments
1 commit
expand
appview/gfi: render gfi label when there isn't any matching issues
expand 0 comments
1 commit
expand
appview/gfi: render gfi label when there isn't any matching issues
expand 2 comments

The error code here should probably be a 500.

1 commit
expand
appview/gfi: render gfi label when there isn't any matching issues
expand 0 comments
1 commit
expand
appview/gfi: render gfi label when there isn't any matching issues
expand 0 comments