search

yannick-cw / notionfy

Syncing kindle highlights to notion
BSD 3-Clause "New" or "Revised" License
138 stars 7 forks source link

deserializartion error - MacOS Catalina & Kindle #9

Closed moccacoke closed 4 years ago

moccacoke commented 4 years ago

sttp.client.DeserializationError notionfys.Notion$.$anonfun$getHighlights$3(Notion.scala:80) scala.util.Either$LeftProjection.map(Either.scala:614) notionfys.Notion$.$anonfun$getHighlights$2(Notion.scala:80) cats.data.Kleisli.$anonfun$flatMap$2(Kleisli.scala:43) cats.effect.internals.IORunLoop$.cats$effect$internals$IORunLoop$$loop(IORunLoop.scala:139) cats.effect.internals.IORunLoop$RestartCallback.signal(IORunLoop.scala:355) cats.effect.internals.IORunLoop$RestartCallback.apply(IORunLoop.scala:376) cats.effect.internals.IORunLoop$RestartCallback.apply(IORunLoop.scala:316) cats.effect.internals.IOShift$Tick.run(IOShift.scala:36) cats.effect.internals.PoolUtils$$anon$2$$anon$3.run(PoolUtils.scala:51) java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128) java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628) java.lang.Thread.run(Thread.java:834) com.oracle.svm.core.thread.JavaThreads.threadStartRoutine(JavaThreads.java:527) com.oracle.svm.core.posix.thread.PosixJavaThreads.pthreadStartRoutine(PosixJavaThreads.java:193)

yannick-cw commented 4 years ago

Hey, thanks for reporting! Are you trying to first time sync or has it worked before? I am releasing a new version 2.0.2 which has a more verbose error logging if you run it with --verbose

Would be great if you can report the result of that. Just make sure to NOT include private information from the response body (e.g. email, full name)

moccacoke commented 4 years ago

Hi,

Good work! It was my first attempt… anyway I managed to solve it by creating a new page in Notion, a blank one…the issue with the first Kindle Highlights page was that it was not empty, I added the toggle with your Setup in it and apparently that was the issue…you need a clean slate page just for the highlights otherwise you will receive the deserialization error…probably notionfy was trying to see if there are other highlights on the page so it not duplicate them (you said that notionfy knows how to check and copy the new highlights…). You got the idea, it got stuck when it found something different formated than one of the regular highlights and it got crashed… my opinion, is good to have it for maybe others that will commit the same mistake I did. Thank you for your workaround, it can be improved as for example summarize all the highlights from the same book in a separate page or table to have things more compact…otherwise is as good as you can get considering is an uniques and fast option to have all your offline highlights transferred…

V/R mocca

On 30 Apr 2020, at 10:38, Yannick Gladow notifications@github.com wrote:

Hey, thanks for reporting! Are you trying to first time sync or has it worked before? I am releasing a new version 2.0.2 which has a more verbose error logging if you run it with --verbose

Would be great if you can report the result of that. Just make sure to NOT include private information from the response body (e.g. email, full name)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/yannick-cw/notionfy/issues/9#issuecomment-621668033, or unsubscribe https://github.com/notifications/unsubscribe-auth/APMR3GE4BWU42OSTYAONLP3RPETHTANCNFSM4MUC7T7A.

yannick-cw commented 4 years ago

Great, #4 is about a different way of organizig the highlights, I am currently looking into that, it is not as trivial :)