Skip to content

Allow maxlag to be explicitly configured instead of hardcoded - #128

Merged
CXuesong merged 1 commit into
CXuesong:masterfrom
mormegil-cz:feature/configurable-maxlag
Oct 12, 2026
Merged

CXuesong merged 1 commit into
CXuesong:masterfrom
mormegil-cz:feature/configurable-maxlag

Conversation

@mormegil-cz

Copy link
Copy Markdown
Contributor

The maxlag parameter is explicitly set on some API calls, but the value is hardcoded to 5 seconds.

This change introduces a configuration option in SiteOptions to set maxlag on all API calls.

@CXuesong CXuesong added this to the v0.9.0 milestone Oct 12, 2026
@CXuesong CXuesong self-assigned this Oct 12, 2026
The maxlag parameter is explicitly set on _some_ API calls, but the value
is hardcoded to 5 seconds.

This change introduces a configuration option in SiteOptions to set
maxlag on all API calls.
@CXuesong
CXuesong force-pushed the feature/configurable-maxlag branch from 8a06701 to 69b6683 Compare October 12, 2026 16:09
@CXuesong
CXuesong merged commit 94b1915 into CXuesong:master Oct 12, 2026
2 checks passed
@CXuesong

Copy link
Copy Markdown
Owner

Thanks for the PR! I merged the change but it may take a while before I revamp the failing unit test cases and release a new package.

@mormegil-cz
mormegil-cz deleted the feature/configurable-maxlag branch October 12, 2026 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants