Skip to content

fix: flatten outputs before building loadingOutputs#3641

Open
i-murray wants to merge 1 commit intoplotly:devfrom
i-murray:bugfix/loading-all-wildcard-3619
Open

fix: flatten outputs before building loadingOutputs#3641
i-murray wants to merge 1 commit intoplotly:devfrom
i-murray:bugfix/loading-all-wildcard-3619

Conversation

@i-murray
Copy link

@i-murray i-murray commented Mar 4, 2026

Summary

Fix dcc.Loading spinner not triggering when a callback Output uses the ALL wildcard.

Problem

In dash-renderer/src/actions/callbacks.ts, loadingOutputs was built with outputs.map(...). When the output spec contains ALL, unwrapIfNotMulti returns resolved outputs as a nested array. Iterating with .map() over this nested structure meant out.id was undefined, so getPath returned undefined and the loading reducer never matched the component path.

Fix

Changed outputs.map(...) to flatten(outputs).map(...) on L794, consistent with how outputs are already flattened on L829 and L912 in the same function.

Fixes #3619

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.

[BUG] dcc.Loading spinner does not trigger when callback Output uses ALL wildcard

1 participant