search

dstotijn / go-notion

Go client for the Notion API.
MIT License
380 stars 39 forks source link

add functionality to list databases #4

Closed benbarten closed 3 years ago

benbarten commented 3 years ago

Adds another client method to list databases as referenced in the API docs: https://developers.notion.com/reference/get-databases.

Adds two unit tests for the above behavior.

dstotijn commented 3 years ago

Hi Ben, thanks for taking the effort to submit this PR, but I think we best don't add this endpoint to the client, following Notion's recommendation (ref) about using the Search endpoint instead. I've thus regarded List Databases as deprecated and deliberately didn't add it to the client.

benbarten commented 3 years ago

I did not see that. Alright, nvm :)