Skip to content

Keep at most one folder/:id/volview_loadable request in flight. - #115

Merged
manthey merged 1 commit into
mainfrom
gate-volview_loadable
Oct 22, 2026
Merged

manthey merged 1 commit into
mainfrom
gate-volview_loadable

Conversation

@manthey

@manthey manthey commented Oct 21, 2026

Copy link
Copy Markdown
Collaborator

The server walks the whole folder subtree to answer, which can take a long time on large trees, and aborting in the browser does not stop that work. Buttons asking about the folder already in flight share its answer, and navigating while a request is in flight only records which folder to check next, so rapid browsing issues at most one further request when the current one finishes. A superseded pending check is dropped, which also keeps a stale answer from updating a newer folder's button.

The server walks the whole folder subtree to answer, which can take a
long time on large trees, and aborting in the browser does not stop that
work. Buttons asking about the folder already in flight share its
answer, and navigating while a request is in flight only records which
folder to check next, so rapid browsing issues at most one further
request when the current one finishes. A superseded pending check is
dropped, which also keeps a stale answer from updating a newer folder's
button.
@manthey
manthey requested a review from PaulHax October 21, 2026 21:13

@PaulHax PaulHax left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@manthey
manthey merged commit 3a25f90 into main Oct 22, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants