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

feat: notion integration #3142

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

umialpha
Copy link

@umialpha umialpha commented Sep 13, 2024

#2894

implements

I followed the pattern WebCrawler, creating the following core components:

  • Adding list/create/delete interfaces to Queryand Mutation.
  • Adding new interface NotionDocumentService and impl NotionServiceImpl
  • Adding list/create/delete interfaces to DBCon,
  • Adding a new enum BackgroundJobEvent.NotionIntregration
  • Adding notion_utils to fetch docs using notion apis.

** I also implemented the frontend part

Next PR

  1. (Possibly required) Merge all kinds of integration jobs into a unified service to avoid the need to add a variable to the web server each time a new service is added.
  2. (Possibly required) Currently, background tasks are executed serially; it may be necessary to change to multi-threaded execution.

@umialpha umialpha changed the title notion integration[WIP] feat: notion integration[WIP] Sep 13, 2024
@umialpha umialpha changed the title feat: notion integration[WIP] feat: notion integration Sep 19, 2024
@umialpha
Copy link
Author

@wsxiaoys ,PTAL, I've finished both the fe and be parts.

@umialpha umialpha force-pushed the roylee/notion-integration branch from 70f78bd to ba4036e Compare September 20, 2024 09:07
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