Page MenuHomeSolus

Update boostnote to 0.15.3
ClosedPublic

Authored by akrenz on Jun 2 2020, 1:32 AM.
Tags
None
Referenced Files
F10923646: D8977.diff
Mon, Jul 10, 12:01 AM
F10851022: D8977.id21588.diff
Jun 12 2023, 8:48 AM
F10847230: D8977.id21634.diff
Jun 11 2023, 9:32 AM
F10827165: D8977.diff
Jun 4 2023, 4:41 AM
F10767545: D8977.diff
May 20 2023, 12:43 AM
F10757002: D8977.id21634.diff
May 17 2023, 6:16 AM
F10733019: D8977.diff
May 11 2023, 7:13 PM
F10724615: D8977.id.diff
May 7 2023, 5:35 PM
Subscribers

Details

Summary

Fixes T8963.

This is an update for the legacy version of boostnote. The newer version mentioned in the Update reuqest was Boostnote.next, which is incompatible with the legacy version of the app. Therefor it should not silently replace the legacy version, but should be added as a new package with "replaces" flag.

Test Plan

Created/edited/deleted Multiple notes and folders. Tried opening an old note storage creatd with the previous version.

Diff Detail

Repository
R4324 boostnote
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

akrenz requested review of this revision.Jun 2 2020, 1:32 AM
JoshStrobl added a subscriber: JoshStrobl.

LGTM, thanks!

P.S. In response to your comment of "Therefor it should not silently replace the legacy version, but should be added as a new package with "replaces" flag." - This is effectively the same as just upgrading this version. The individual isn't going to get an option during upgrade to not replace it. I think either some heads up should be given, or we should figure out a way to ship both of them as co-installable (though that'd involve considerable effort to change install paths, patch config paths, etc.).

That being said, I'd like to know what is actually incompatible. At least according to this issue, they're just using a file hierarchy and leveraging CSON in the "legacy" implementation. Was this changed in the "next" version to something which leverages a database (sqlite, indexeddb, drivers for other stuff like postgresql, etc.)?

This revision is now accepted and ready to land.Jun 4 2020, 12:56 PM
This revision was automatically updated to reflect the committed changes.