Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

[Usntructured loaders] Make Unstructured API URL optional when environment variable is present #3414

Merged
merged 3 commits into from
Nov 1, 2024
Merged

[Usntructured loaders] Make Unstructured API URL optional when environment variable is present #3414

merged 3 commits into from
Nov 1, 2024

Conversation

JJK801
Copy link
Contributor

@JJK801 JJK801 commented Oct 25, 2024 •
edited
Loading

Hi Flowise,

When having an UNSTRUCTURED_API_URL set, we don't want this params as required in loaders.

This PR aims to:

  • make parameter required when env var is not set
  • make parameter optional when env var is set
  • allow parameter to override when env var is set

@JJK801 JJK801 force-pushed the fix/unstructured-required-fields branch from f8f198a to ee2b1ac Compare November 1, 2024 20:06
@HenryHengZJ HenryHengZJ merged commit a159da5 into FlowiseAI:main Nov 1, 2024
2 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@HenryHengZJ HenryHengZJ HenryHengZJ approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@JJK801 @HenryHengZJ