Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: refactor Notion database querying methods #15

Merged
merged 1 commit into from
Jan 24, 2024
Merged

Conversation

kkdai
Copy link
Owner

@kkdai kkdai commented Jan 24, 2024

  • Add a new method QueryContainsDatabase to query the Notion database by property and value
  • Move the notionapi.NewClient initialization to a separate variable client
  • Modify the QueryContainsDatabase method to add the UId to the filter conditions
  • Add a new method QueryDatabaseContainsByName to query the Notion database by name and UId
  • Add a new method QueryDatabaseContainsByEmail to query the Notion database by email and UId
  • Add a new method QueryDatabaseContainsByTitle to query the Notion database by title and UId
  • Add a new method QueryDatabaseContains to query the Notion database by name, email, and title

- Add a new method `QueryContainsDatabase` to query the Notion database by property and value
- Move the `notionapi.NewClient` initialization to a separate variable `client`
- Modify the `QueryContainsDatabase` method to add the `UId` to the filter conditions
- Add a new method `QueryDatabaseContainsByName` to query the Notion database by name and UId
- Add a new method `QueryDatabaseContainsByEmail` to query the Notion database by email and UId
- Add a new method `QueryDatabaseContainsByTitle` to query the Notion database by title and UId
- Add a new method `QueryDatabaseContains` to query the Notion database by name, email, and title

Signed-off-by: Evan Lin <evan.if.lin@linecorp.com>
@kkdai kkdai merged commit 8fe02f6 into main Jan 24, 2024
1 check passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant