chore(code): github branch picker loading all branches#1592
chore(code): github branch picker loading all branches#1592VojtechBartos merged 5 commits intomainfrom
Conversation
a42284e to
9c24c33
Compare
|
i don't think it's your fault haha i'm just curious - where is that latency coming from? looked like at least a few seconds before we got the first page of results - is that posthog api or github api taking so long? can be a followup, but if there's no way around that, we should consider some local caching here so it's snappier most of the time |
| ); | ||
|
|
||
| // Auto-fetch remaining pages in background once the first page arrives | ||
| useEffect(() => { |
There was a problem hiding this comment.
can we abort this query once the user selects a branch?
or if we keep this and fetch everything even after the user is done, we should at least be caching the results (see global comment)
|
@adboio soooo :) ...
the changes i have made
WDYT? |
skoob13
left a comment
There was a problem hiding this comment.
LGTM. Can we add the search? How would we find master for PostHog's monorepo?
|
@skoob13 there is UI based search, so you are going to find it. There is a small issue related to that on API side, so fixing it |
Problem
The branches API was capped at 2000 branches. Repositories with more than that were missing branches in the selector.
Changes
Prerequisite
Posthog API changes PostHog/posthog#53931
Showcase
code-paginating.branches.mov