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

[DRAFT] chore: update Firecrawl version and add FirecrawlExtractTool #4073

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

Conversation

ftonato
Copy link

@ftonato ftonato commented Feb 24, 2025

Hello,

I am one of the people building Firecrawl, and it would be great to update the version to support our new API version (v1). I made a few changes, but I wasn’t able to test them.


It would be helpful if we could test it before merging to ensure everything is working as expected. Could someone help me understand how to test it?

The visual part is working as expected, but my problem is how to check the request/response from each method we have?

image

Thank you!

@HenryHengZJ
Copy link
Contributor

thanks @ftonato !

One way to test it is via Document Store:
image
https://docs.flowiseai.com/using-flowise/document-stores

optional: true,
additionalParams: true
}
// ... (other input parameters)
]
this.outputs = [
{
Copy link
Contributor

Choose a reason for hiding this comment

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

We can't change the output here, as they are meant to connect to other nodes:
image

Copy link
Author

Choose a reason for hiding this comment

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

I will revert this part!

@ftonato
Copy link
Author

ftonato commented Feb 27, 2025

Hey @HenryHengZJ,

I am still trying this approach, but every time I get the same error: Failed to preview chunks: Error: documentStoreServices.previewChunksMiddleware - Error: documentStoreServices.previewChunks - Error: documentStoreServices.splitIntoChunks - Request failed with status code 400. In the network tab, I can see that I have a 500 error.

Let me share the video I recorded so you can see what I’m doing:

CleanShot.2025-02-27.at.23.42.59.mp4

@ftonato
Copy link
Author

ftonato commented Mar 4, 2025

Hello @HenryHengZJ, did you have the chance to see my last message?

@ftonato
Copy link
Author

ftonato commented Mar 13, 2025

Hello @HenryHengZJ, I apologize for pinging you again, but I am still waiting for some clarity. Could you point me to someone who can help me with this?

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