Skip to content

fix(editor): save all extra settings changed in quick succession - #3873

Merged
AndyScherzinger merged 1 commit into
mainfrom
fix/editor-extra-settings-quick-changes
Nov 11, 2026
Merged

AndyScherzinger merged 1 commit into
mainfrom
fix/editor-extra-settings-quick-changes

Conversation

@AndyScherzinger

Copy link
Copy Markdown
Member

Split out of #3833, so it can be merged and backported on its own. The commits are unchanged and drop out of #3833 once this is merged.

  • Extra settings changed in quick succession are all saved; switching on "Shuffle options" and "Add other" quickly saved only "Add other".

Tests

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

The extra settings of a question are saved debounced, but only the
change of the last call within the debounce time was saved, merged into
the settings from before the first one. Switching on "Shuffle options"
and "Add other" quickly saved only "Add other". The changes are now
collected and saved together.

Assisted-by: Claude Code:claude-opus-5-5
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
@AndyScherzinger AndyScherzinger added bug Something isn't working 3. to review Waiting for reviews labels Nov 11, 2026
@AndyScherzinger AndyScherzinger self-assigned this Nov 11, 2026
@Chartman123

Copy link
Copy Markdown
Collaborator

/backport to stable5.4

@backportbot backportbot Bot added the backport-request Pending backport by the backport-bot label Nov 11, 2026
@AndyScherzinger
AndyScherzinger merged commit 95f3213 into main Nov 11, 2026
42 checks passed
@AndyScherzinger
AndyScherzinger deleted the fix/editor-extra-settings-quick-changes branch November 11, 2026 09:07
@backportbot backportbot Bot removed the backport-request Pending backport by the backport-bot label Nov 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants