Skip to content

batches: Fix diff unavailable after step finishes

Warren Gifford requested to merge rc/diff-unavailable into main

Created by: Piszmog

Closes #38600 (closed).

The callback queryFileDiffs will only update if there are changes to the dependencies. When a step has a state change (for example finishes), this function does not get re-ran since none of the dependencies changed.

I decided to pass the whole step instead of the step number. This way, then the step has changes (for example finishes), this would cause the useCallback to rerun.

Test plan

Manual testing (see video)

https://user-images.githubusercontent.com/38407415/180518420-d870fa39-6d62-41b2-a842-5a3742ca8c8f.mov

App preview:

Check out the client app preview documentation to learn more.

Merge request reports

Loading