tangled.org / core
Monorepo for Tangled tangled.org
fork atom

appview: add issue search endpoint #496

merged opened by boltless.me targeting master from boltless.me/core: feat/search
STACK 13
#711 appview/notify: merge NewPullClosed/Merged/Reopen to NewPullState
1
#6
#710 appview: add pull_reopen event
0
#6
#709 appview/pulls: add search endpoint
0
#6
#708 appview/indexer: add pulls indexer
0
#6
#707 appview: add issue_reopen event
0
#6
#706 appview/indexer: put bleve index dir under indexes/ dir
0
#6
#693 appview/notify: pass logger with mergedLogger
2
#9
#695 appview/indexer: handle NewIssueClosed
1
#9
#694 appview/notify: add DeleteIssue event
1
#9
#496 appview: add issue search endpoint
5
#13
#495 wip: add indexer mappings
1
#14
#494 appview: add basic issue indexer (wip)
2
#14
#685 go.mod: add bleve as a dependency
1
#10
Labels
enhancement
assignee

None yet.

Participants 2
AT URI
at://did:plc:xasnlahkri4ewmbuzly2rlc5/sh.tangled.repo.pull/3lwecnvoz5422
+2 -2
Interdiff #12 → #13
appview/db/issues.go

This file has not been changed.

+2 -2
appview/issues/issues.go
··· 798 798 l.Error("failed to search for issues", "err", err) 799 799 return 800 800 } 801 - l.Debug("searched issues with indexer", "res.Hits", res.Hits) 802 801 ids = res.Hits 802 + l.Debug("searched issues with indexer", "count", len(ids)) 803 803 } else { 804 804 ids, err = db.GetIssueIDs(rp.db, searchOpts) 805 805 if err != nil { 806 806 l.Error("failed to search for issues", "err", err) 807 807 return 808 808 } 809 - l.Debug("indexed all issues from the db", "ids", ids) 809 + l.Debug("indexed all issues from the db", "count", len(ids)) 810 810 } 811 811 812 812 issues, err := db.GetIssues(
appview/pages/templates/repo/issues/issues.html

This file has not been changed.

History

14 rounds 5 comments
sign up or login to add to the discussion
boltless.me submitted #13
diff
1 commit
expand
d07fbce2
appview: add issue search endpoint
expand 0 comments
pull request successfully merged
boltless.me submitted #12
diff interdiff
1 commit
expand
17e2a3bb
appview: add issue search endpoint
expand 0 comments
boltless.me submitted #11
diff interdiff
1 commit
expand
804963e6
appview: add issue search endpoint
expand 0 comments
boltless.me submitted #10
diff interdiff
1 commit
expand
3d998c0f
appview: add issue search endpoint
expand 0 comments
boltless.me submitted #9
diff interdiff
1 commit
expand
7549edf4
appview: add issue search endpoint
expand 0 comments
boltless.me submitted #8
diff interdiff
1 commit
expand
1a4bb573
appview: add issue search endpoint
expand 0 comments
boltless.me submitted #7
diff interdiff
1 commit
expand
1a4bb573
appview: add issue search endpoint
expand 0 comments
boltless.me submitted #6
diff interdiff
1 commit
expand
36b591d7
appview: add issue search endpoint
expand 0 comments
boltless.me submitted #5
diff interdiff
1 commit
expand
36b591d7
appview: add issue search endpoint
expand 3 comments
oppi.li

lgtm

oppi.li

do we need the log here? i believe that log message should be changed atleast because indexing does not occur there.

boltless.me

I can change the wording a bit. The intention was to log the list of issue ids gathered from one of two methods (indexer/db)

boltless.me submitted #4
diff interdiff
1 commit
expand
2d3f191b
appview: add issue search endpoint
expand 0 comments
boltless.me submitted #3
diff interdiff
1 commit
expand
34b87183
appview: add issue search endpoint
expand 0 comments
boltless.me submitted #2
diff interdiff
1 commit
expand
36bbfc8b
appview: add issue search endpoint
expand 1 comment
boltless.me submitted #1
diff interdiff
1 commit
expand
73c72bea
appview: add issue search endpoint
expand 0 comments
boltless.me submitted #0
diff
1 commit
expand
54342b94
appview: add issue search endpoint
expand 1 comment
boltless.me

Indexer mappings/tokenization rules have some room to be improved, but this should be enough as a MVP for now.