search

kjk / notionapi

Unofficial Go API for Notion.so
https://blog.kowalczyk.info/article/c9df78cbeaae4e0cb2848c9964bcfc94/using-notion-api-go-client.html
BSD 2-Clause "Simplified" License
1.82k stars 86 forks source link

no users when trying to resolve collection_view #29

Closed caarlos0 closed 4 years ago

caarlos0 commented 4 years ago

notion did a maintenance yesterday, and I think they changed something that broke the lib.

https://twitter.com/NotionStatus/status/1305691756353126400

my daily refresh now fails with this error:

https://github.com/caarlos0/carlosbecker.com/runs/1120562268?check_suite_focus=true

I don't know enough of the API to attempt a fix... but I could debug it to the check that's failing: this one

Thanks for this lib BTW :) awesome work!

caarlos0 commented 4 years ago

BTW: removing the if and passing a built user to the queryCollection method seems to resolve the issue... will work on a PR :)

kjk commented 4 years ago

Should be fixed by https://github.com/kjk/notionapi/commit/c95813fdf9a5e91b6b7845a50ee253024a07b021

caarlos0 commented 4 years ago

thanks!