Tom French
Tom French
Cool, yeah no rush on this so feel free to merge whenever.
Let's park this pending https://github.com/actualbudget/actual/pull/1077 as the extra types will be useful when fixing this test. Looks like we've been loose with the definition of a message here.
> Something to consider cherry-picking: [5b23505](https://github.com/actualbudget/actual/commit/5b23505783fb94deae2a77a966bbc8cebdda7e3b) Thanks, I noticed that the issue was being hidden by the `any` return types of `send/recv` so this helped get TS to spot this...
I don't think I'll be able to meaningfully progress this over the next month or so. I can't run the tests locally on my laptop so I would need to...
@j-f1, please feel free. I'm happy to help review any further changes but my laptop can't really handle proper debugging of Actual.
Marking draft as I rebased this onto master without noticing that it depends on types from #1077.
Thanks for working on this @Shazib. I much prefer to use the desktop app and really struggled to get it building in CI so glad to see it working again.
This PR would be very helpful for us running cross-platform tests using zx in CI.
Related to #462
@Altrue @tien Ah, thank you. I must admit I didn't test that as a required ordering to fields in JSON doesn't really make sense. I can't quite remember why I...