dominiksta / wournal

Simple "digitial paper" for note taking and PDF annotation. Heavily inspired by Xournal.
GNU Lesser General Public License v2.1
44 stars 5 forks source link

Error: window.setZoom After Initial Start of Wournal #8

Open qhga opened 1 month ago

qhga commented 1 month ago

Error happens right after the initial start of version 0.0.5 after the update, on cosecutive starts the error does not happen. Wournal still functions normal after the error message is dismissed:

{
  "buildInfo": {
    "wournalVersion": "0.0.5",
    "gitVersion": "bd853c1",
    "buildTime": "2024-05-22T13:00:08.948Z",
    "production": true,
    "userAgent": "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) wournal/0.0.5 Chrome/118.0.5993.159 Electron/27.1.3 Safari/537.36"
  },
  "error": {
    "name": "Error",
    "constructor": "Error",
    "message": "Error invoking remote method 'window:setZoom': TypeError: Error processing argument at index 0, conversion failure from undefined",
    "stack": [
      "Error: Error invoking remote method 'window:setZoom': TypeError: Error processing argument at index 0, conversion failure from undefined"
    ]
  },
  "logs": [
    "2024-05-22T13:48:41.416Z [info]: Logging initialized",
    "2024-05-22T13:48:41.418Z [info]: got ConfigDTO version 0.5, wanted 0.8",
    "2024-05-22T13:48:41.418Z [info]: Versions to Update: 0.6,0.7,0.8",
    "2024-05-22T13:48:41.418Z [info]: updating ConfigDTO to version 0.6",
    "2024-05-22T13:48:41.418Z [info]: updating ConfigDTO to version 0.7",
    "2024-05-22T13:48:41.418Z [info]: updating ConfigDTO to version 0.8",
    "2024-05-22T13:48:41.421Z [debug]: Undo stack cleared"
  ]
}
dominiksta commented 1 week ago

Thank you for bringing this to my attention. I just checked and at least this does not happen on a fresh install of Wournal, only after an upgrade. But for the next release, I will try to fix this.