Search Race Conditions

[dcramer]

There's an issue with the way we're relying on debouncing right now for all networked searches.

Basically assume two calbacks (since this is an onKeyUp handler):

• t1 = callback() executes
• t2 = callback() gets debounced
• t2 = callback() executes
• t2 = callback() finishes
• t1 = callback() finishes

This ends up showing the results of t1 when you want t2

[dcramer]

There's two key areas where this problem exists right now:

• SelectDialog
• SearchPanel

I've improved both, but they still end up w ith writing outdated results when you're still typing a query. I dont think either has the actual race condition anymore, but keeping this open for now until I fix the entire implementation.

[dcramer]

This is now fixed. Issue was I needed to wrap the callbacks passed into useDebounceCallback with useCallback (or useMemo).

Still want to improve the experience, but it doesnt feel buggy anymore.