yannick-cw / notionfy

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

HTTP request failed with status code: 400 #20

Open re-zhe opened 3 years ago

re-zhe commented 3 years ago

Hi,

Thanks a lot for amazing project! I have installed the latest version via Homebrew and got the following error:

Parsed kindle highlights to internal format Fetching highlights from Notion...

Http request failed with status code: 400 and body: {"errorId":"301bd401-06f9-4526-9c67-f89547582591","name":"ValidationError","message":"Invalid input."}

I am using macOS 11.2.2. Could you tell me is there a way to fix this issue?

yannick-cw commented 3 years ago

I’ll have a look of this is a general issue at the moment, thanks.

On Thu 11. Mar 2021 at 11:13, re-zhe @.***> wrote:

Hi,

Thanks a lot for amazing project! I have installed the latest version via Homebrew and got the following error:

Parsed kindle highlights to internal format Fetching highlights from Notion...

Http request failed with status code: 400 and body: {"errorId":"301bd401-06f9-4526-9c67-f89547582591","name":"ValidationError","message":"Invalid input."}

I am using macOS 11.2.2. Could you tell me is there a way to fix this issue?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/yannick-cw/notionfy/issues/20, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAS636YQOFYXW7X2AZPOO2DTDCJWNANCNFSM4Y77H33Q .

gpks commented 3 years ago

I have the same problem and I think here is the solution: https://github.com/jamalex/notion-py/issues/292

Here you have also used limit: 10000. Thanks for building this tool!!

yannick-cw commented 3 years ago

I’ll have a look next week after vacation, thanks for the fix suggestion!

On Sat 3. Apr 2021 at 16:31, Małgorzata Ksionek @.***> wrote:

I have the same problem and I think here is the solution: jamalex/notion-py#292 https://github.com/jamalex/notion-py/issues/292

Here you have also used limit: 10000. Thanks for building this tool!!

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/yannick-cw/notionfy/issues/20#issuecomment-812873351, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAS6365BQAVXHBFWY2SULS3TG4RFXANCNFSM4Y77H33Q .

yannick-cw commented 3 years ago

Could you try with the new version?

gpks commented 3 years ago

I was using homebrew and I cannot upgrade to 0.3.1 via this method :(

yannick-cw commented 3 years ago

Ah I missed to bump the version in the app! Sorry, the newest brew version should work anyway even if it is saying 0.3.0

On Tue 6. Apr 2021 at 22:48, Małgorzata Ksionek @.***> wrote:

I was using homebrew and I cannot upgrade to 0.3.1 via this method :(

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/yannick-cw/notionfy/issues/20#issuecomment-814430021, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAS636ZNXCENLQTAUQNIER3THNXRLANCNFSM4Y77H33Q .

pk478 commented 3 years ago

Just tried on 0.3.0 and still have the issue. Thanks for creating and maintaining the tool !

yannick-cw commented 3 years ago

Ahh okay seems like the new limit is actually 100. I can quick fix it to use 100 but if you have more then 100 highlights this would need pagination now and is a bit more difficult.

yannick-cw commented 3 years ago

Now 0.3.1 should be available via brew and work with the limit of 100. Means if you have too many highlights it will duplicate them and append them in Notion on each Sync. Workaround: Delete all highlights before a sync, this will do a fresh sync then. I hope I can fix it with pagination in the future.

pk478 commented 3 years ago

Upgraded to 0.3.1, works perfectly for me now. Thank you !

kierglover commented 3 years ago

Can verify as well that after updating to 0.3.1 it's working. Thanks again for the maintenance of this 🙏