Monorepo for Tangled โ€” https://tangled.org

appview: add basic issue indexer (wip) #494

merged
opened by boltless.me targeting master from boltless.me/core: feat/search
  • Heavily inspired by gitea
  • add GetAllIssues which only receives a paginator and gathers all issues ignoring repoAt field

Signed-off-by: Seongmin Lee boltlessengineer@proton.me

0
by boltless.me 0 comments
expand 1 commit
appview: add basic issue indexer (wip)
sign up or login to add to the discussion
1
by boltless.me 0 comments
expand 1 commit
appview: add basic issue indexer
sign up or login to add to the discussion
2
by boltless.me 0 comments
expand 1 commit
appview: add basic issue indexer
sign up or login to add to the discussion
3
by boltless.me 1 comment
expand 1 commit
appview: add basic issue indexer
  • this is an N+1 query, because we first get all issues and then make one query for each issue. understandably this only needs to happen when populating the indices but we could improve this by just making one query
  • we can use a pagination free API for getting issues with comments while populating indexer
  • more of a note to self: search index should also handle issue deletions
sign up or login to add to the discussion
4
by boltless.me 0 comments
expand 1 commit
appview: add basic issue indexer
sign up or login to add to the discussion
5
by boltless.me 0 comments
expand 1 commit
appview: add basic issue indexer
sign up or login to add to the discussion
6
by boltless.me 1 comment
expand 1 commit
appview: add basic issue indexer

this changeset largely lgtm!

sign up or login to add to the discussion
7
by boltless.me 0 comments
expand 1 commit
appview: add basic issue indexer
sign up or login to add to the discussion
8
by boltless.me 0 comments
expand 1 commit
appview: add basic issue indexer
sign up or login to add to the discussion
9
by boltless.me 0 comments
expand 1 commit
appview: add basic issue indexer
sign up or login to add to the discussion
10
by boltless.me 0 comments
expand 1 commit
appview: add basic issue indexer
sign up or login to add to the discussion
11
by boltless.me 0 comments
expand 1 commit
appview: add basic issue indexer
sign up or login to add to the discussion
12
by boltless.me 0 comments
expand 1 commit
appview: add basic issue indexer
sign up or login to add to the discussion
13
by boltless.me 0 comments
expand 1 commit
appview: add basic issue indexer
sign up or login to add to the discussion
14
by boltless.me 0 comments
expand 1 commit
appview: add basic issue indexer
pull request successfully merged
sign up or login to add to the discussion
Labels

None yet.

assignee

None yet.

Participants 2
AT URI
at://did:plc:xasnlahkri4ewmbuzly2rlc5/sh.tangled.repo.pull/3lwecnvoz4z22