bdring / FluidNC

The next generation of motion control firmware
Other
1.61k stars 383 forks source link

Improve web request handling #1370

Closed michmela44 closed 1 week ago

michmela44 commented 1 week ago

Hold State Requests

Pull request #1334 introduced a new screen during the Hold state that only allows you to Resume or Stop the current job.

image

This PR reverts back to the previous behavior which will allowed it to serve up file requests when inMotionState() is false, even in Hold state.

This is in agreement to the screen that shows when in motion, that appears to say that issuing a feedhold is recommended to be able to reload the WebUI.

image

304 responses

This PR also adds a check when in motion that will return a 304 when the correct ETag is received and the hash of the file is already cached in memory.

This should reduce the amount of times that the cut down UI is seen if the file needs to be retrieved again for some reason.