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

Allow more features from API endpoint #1

Merged
merged 3 commits into from
Feb 15, 2024

Conversation

telekosmos
Copy link

We needed to get exported a whole bunch of blocks children along with the parent to embed the documentation in a vector store. So, starting off you work we implemented chance to perform:

  • recursive export
  • enable/disable to export PDFs and images

Also, added a new method to unzip all exported files into a folder (for us that is useful to use LangChain's NotionLoader

Guille added 3 commits February 9, 2024 11:04
Added two features which are allowed by the Notion endpoint API and can be useful to further
customize the export:
- recursive, allows to export one block and alla the block children
- no-files, to avoid downloading files like pdf or images (as it is optional from the UI)

Also, addeda function `getMdFiles` to dump the files in a folder (this is useful if we
want to use a function from langchain, for example, to load all files in a folder).
Copy link
Collaborator

@UlisesGascon UlisesGascon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Comment on lines +49 to +50
timeZone: "Europe/Zurich",
locale: "en",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we can allow in the future that these values are configurable? 🤔

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yep, I'll try to configure it in the next PR, event if it is mixed with the package qualified name

@UlisesGascon UlisesGascon merged commit 0c75d31 into onebeyond:master Feb 15, 2024
2 checks 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.

2 participants