joel1st
joel1st
Yeah definitely feels overkill - the alternative (before the PR gets accepted) is 30/40 minutes of intermittent connectivity - which isn't acceptable for prod. For the meantime we've scripted a...
Thanks for the reply @phryneas . You are correct in that it revolves around that statement. From the redux docs I linked, this is the example: ``` try { const...
@phryneas Thanks for the thorough explanation, I can appreciate trying to maintain consistent behaviour. I guess I have just associated nested try / catches as a potential code smell, especially...
Thanks @phryneas - The code snippets we have been discussing have been pessimistic updates (they occur after the successful `queryFulfilled`), which will potentially not be followed by a server sync...
@phryneas - oh I completely agree, the likelihood of bugs happening here should be very low. It's more of a peace of mind thing and not second guessing when there...
@matheussampaio - I'm running into the same issue, I've tracked the regression back to this commit: https://github.com/orval-labs/orval/commit/d2baa511bc7600087d509273f79936f9f71fef58 The good news is that it only effects the latest version 7.10.0, if...