Desktop: Downgrade to Electron 35.7.5#13013
Conversation
|
Thanks for the update. I guess we can then move to Electron 37 from v3.5 so that there's plenty of time to beta test it. |
|
FWIW for the Linux users, the snap has had 15,000 updates for 3.4.6 in the past 48 hours; whilst testing in stable isn't ideal, through sheer numbers alone I'd argue that's substantially more testing than the beta releases would have through numbers alone. If there's further releases in the 3.4 series then there's a chance I'll revert this commit for the snap specifically because more people have used the newer Electron than the older one IMO and the regressions aren't applicable (e.g., Windows title bar) or will happen anyway down the line (WSL2) - the next Electron version bump is likely to include Wayland by default (Electron 39) so I think there's some value in having Electron 37 decently tested for Joplin 3.5 so there's not too much at once. Let me know if that sounds too much but I think given realistic risk modelling, it's a decent compromise given the current scenario. |
That seems reasonable. |
|
Will rolling back cause errors for people who are already on v3.4.6? (https://github.com/microsoft/winget-pkgs/tree/master/manifests/j/Joplin/Joplin/3.4.6) |
|
No, data formats are heavily independent of Electron. |
Summary
This pull request reverts to the 35.x.x branch of Electron. #12951 upgraded from Electron 35.5.1 to Electron 37.3.0. The primary reason for including this upgrade in Joplin v3.4 was probably to ensure that this security patch (also in Electron 35.6.0) was applied.
This pull request downgrades to Electron 35.7.5 and thus should still include electron/electron#47369.
Notes
Possible benefits of this pull request
Drawbacks
Testing
Manual testing is currently in progress. Additional automated integration tests should run in CI (however, with a dev build of Joplin).
Plan: The following should be tested on each platform:
On Windows:
The following should be tested on at least one platform:
Test results:
Note: Post-merge, I have checked that the Windows and MacOS applications can be built, installed, and started.