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

[BUG] Document Stores - Notion Database: "Please fill in mandatory fields." #3847

Open
BadrLaajali opened this issue Jan 10, 2025 · 1 comment
Labels
bug Something isn't working

Comments

@BadrLaajali
Copy link

Describe the bug
When trying to save a Notion Document Store Loader the application says
"Please fill in mandatory fields."
even if all mandatory fields are set.

To Reproduce
Steps to reproduce the behavior:

Go to Document Stores
Click on a newly created Store
Click on Add Document Loader
Click on Notion DataBase
Fill in all mandatory fields
For the first time is working so data will be insert
Go back again and select in Option Preview & Process
Select Preview or Process and See error "Please fill in all mandatory fields."

Expected behavior
Expect to load data again from database

Screenshots
CleanShot 2025-01-10 at 10 16 34@2x

Setup

  • Installation: docker
  • Flowise Version: 2.2.3
  • OS: Linux
  • Browser: Arc
@HenryHengZJ HenryHengZJ added the bug Something isn't working label Jan 11, 2025
@saatchi-david
Copy link
Contributor

saatchi-david commented Jan 15, 2025

I'm experiencing this as well.
Latest version.
OS: Win 11
Browser: Chrome

Hi @HenryHengZJ , if I may add here, it seems there's also an option missing from the doc loader.
I mentioned this before, but thought it was odd that the Notion db properties get upserted both as metadata as well as in the body of the document. In most cases, this isn't ideal as the same data gets sent to the LLM twice, and we don't have control for omitting it.

I took a look at the langchain docs and it seems there's an option to address this:

propertiesAsHeader: true, // Prepends a front matter header of the page properties to the page contents

Found here:
https://js.langchain.com/docs/integrations/document_loaders/web_loaders/notionapi/

When addressing this bug, would it be possible to add this option?

Greatly appreciated as always.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants