Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

refactor: remove UID argument from NotionDB methods #17

Merged
merged 2 commits into from
Jan 25, 2024
Merged

Conversation

kkdai
Copy link
Owner

@kkdai kkdai commented Jan 25, 2024

  • Added UID field to the NotionDB struct
  • Changed the QueryDatabaseContains method to not take UID as input
  • Changed the QueryDatabaseByEmail method to not take UID as input
  • Changed the AddPageToDatabase method to not take UID as input
  • Removed the UId argument from the QueryDatabase method
  • Removed the UId argument from the QueryContainsDatabase method
  • Removed the UId argument from the QueryDatabaseByName method
  • Removed the UId argument from the QueryDatabaseContainsByName method
  • Removed the UId argument from the QueryDatabaseContainsByEmail method
  • Removed the UId argument from the QueryDatabaseContainsByTitle method
  • Removed the UId argument from the QueryDatabaseContains method

kkdai added 2 commits January 25, 2024 10:28
- Added `UID` field to the `NotionDB` struct
- Changed the `QueryDatabaseContains` method to not take `UID` as input
- Changed the `QueryDatabaseByEmail` method to not take `UID` as input
- Changed the `AddPageToDatabase` method to not take `UID` as input
- Removed the `UId` argument from the `QueryDatabase` method
- Removed the `UId` argument from the `QueryContainsDatabase` method
- Removed the `UId` argument from the `QueryDatabaseByName` method
- Removed the `UId` argument from the `QueryDatabaseContainsByName` method
- Removed the `UId` argument from the `QueryDatabaseContainsByEmail` method
- Removed the `UId` argument from the `QueryDatabaseContainsByTitle` method
- Removed the `UId` argument from the `QueryDatabaseContains` method

Signed-off-by: Evan Lin <[email protected]>
@kkdai kkdai merged commit b1f0b06 into main Jan 25, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant