Skip to content

Increase populate timeout to 30s#1935

Merged
leoniedickson merged 3 commits into
mainfrom
issue/1930
May 25, 2026
Merged

Increase populate timeout to 30s#1935
leoniedickson merged 3 commits into
mainfrom
issue/1930

Conversation

@leoniedickson
Copy link
Copy Markdown
Collaborator

Handles transient server slowness (e.g. cold HAPI caches on first use of the day) by automatically retrying a failed populate rather than immediately showing "Form not populated" to the user.

Handles transient server slowness (e.g. cold HAPI caches on first use
of the day) by automatically retrying a failed populate rather than
immediately showing "Form not populated" to the user.
@leoniedickson leoniedickson linked an issue May 11, 2026 that may be closed by this pull request
Increases the populate timeout to 30 seconds to accommodate HAPI server
warm-up on first use of the day, avoiding the 10-second timeout that
caused silent population failures requiring manual retry.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@leoniedickson leoniedickson changed the title Retry population once on first attempt failure Increase populate timeout to 30s May 14, 2026
@leoniedickson leoniedickson requested a review from clinnygee May 18, 2026 03:49
Copy link
Copy Markdown
Collaborator

@clinnygee clinnygee left a comment

Choose a reason for hiding this comment

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

Looks like the change makes sense to me, obviously pretty hard to test though whether it helps with the issue.

@leoniedickson leoniedickson merged commit 6c0c45f into main May 25, 2026
12 checks passed
@leoniedickson leoniedickson deleted the issue/1930 branch May 25, 2026 02:35
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.

Not populating on first load

2 participants