-
Notifications
You must be signed in to change notification settings - Fork 88
Open
Description
First of all,
thank you for the repo! I am very much enjoying it.
Currently, I am testing some workloads with Search.Do function that requires a SearchFilter.
Notion API does not require for us to specify any filter (basically omit,) but I think there must be other reasons you have enforced it not to be nil. However, the objects we retrieve from the Search.Do function do provide the ObjectType, which could hint us the type of the object, meaning that we could possibly skip the SearchFilter if that is nil.
So... if there's any chance, could I post PR with the following and the some tests:
- SearchFilter can be nil
- Search.Do() will no longer reference nil pointer that were supposed* to be a SearchFilter instance.
But as always, I would like to learn the reason.
Thank you!
Metadata
Metadata
Assignees
Labels
No labels