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
+3 -3
Interdiff #9 → #10
appview/db/issues.go

This file has not been changed.

+3 -3
appview/issues/issues.go
··· 792 792 var ids []int64 793 793 searchOpts := models.IssueSearchOptions{ 794 794 Keyword: keyword, 795 - RepoAt: f.RepoAt().String(), 796 - IsOpen: isOpen, 797 - Page: page, 795 + RepoAt: f.RepoAt().String(), 796 + IsOpen: isOpen, 797 + Page: page, 798 798 } 799 799 if keyword != "" { 800 800 res, err := rp.indexer.Search(r.Context(), searchOpts)
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 interdiff
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
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.