-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
feat: Add Notion Sync #2878
base: main
Are you sure you want to change the base?
feat: Add Notion Sync #2878
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
189efe3
to
f002a09
Compare
loop.run_until_complete(fetch_and_store_notion_files_async(access_token, user_id)) | ||
|
||
|
||
async def fetch_and_store_notion_files_async(access_token: str, user_id: UUID): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe needs to be moved to a separate file. Will be done in merge with feature branch
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM :+1 Nice work on a huge PR :)
backend/core/poetry.lock
Outdated
@@ -1872,6 +1872,9 @@ docs = ["myst-parser", "pydata-sphinx-theme", "sphinx-autodoc-typehints", "sphin | |||
test = ["ipykernel", "pre-commit", "pytest (<8)", "pytest-cov", "pytest-timeout"] | |||
|
|||
[[package]] | |||
name = "langchain" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
poetry lock
This pull request adds functionality to sync with Notion. It includes the following changes:
Added authentication for Notion
Fixed syncing credentials for Notion
Added listing of Notion items
Added support for syncing with Notion in the sync service
Added notification to Notion