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

appview: implement pagination for pipelines index #521

closed
opened by ptr.pet targeting master from ptr.pet/core: pipeline-paginated
0
by ptr.pet 1 comment
expand 1 commit
appview: implement pagination for pipelines index

on the whole this looks good, FilterBetween("row_num"...) is a bit magic because it is not possible to tell at a glance that row_num is a column of pipelines, but until we land on a good mechanism for a generic interface for paginating db responses, this is perfect.

thanks for working on this!

sign up or login to add to the discussion
1
by ptr.pet 1 comment
expand 1 commit
appview: implement pagination for pipelines index

had to rename filter to Filter since i wanted to use it elsewhere, but otherwise mostly the same aside from the changes from #519

sign up or login to add to the discussion
2
by ptr.pet 0 comments
expand 1 commit
appview: implement pagination for pipelines index
sign up or login to add to the discussion
3
by ptr.pet 1 comment
expand 1 commit
appview: implement pagination for pipelines index

i think i'd like to rework this from scratch, unfortunately theres been plenty of codechurn around this part of the code over the past few weeks.

closed without merging
sign up or login to add to the discussion
Labels
enhancement
performance
assignee

None yet.

Participants 2
AT URI
at://did:plc:dfl62fgb7wtjj3fcbb72naae/sh.tangled.repo.pull/3lwubheua2x22