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: Support Google Cloud Storage #4061

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

Conversation

AllenFang
Copy link

hi team, I would like to make Flowise to support GCS as one of storage type so please help to me review this PR. I've been tested my work a lots, if you need a demo as the screenshot or screen recording, please let me know.

furthermore, if i'm lucky that this MR could be merged. I will help to refactor the code a bit especially the storageUtil.ts.

@AllenFang
Copy link
Author

hi @0xi4o boss, hold on.. lol, I found that I missing the logs
https://github.com/FlowiseAI/Flowise/blob/main/packages/server/src/utils/logger.ts#L16

I will patch this MR in these couple of days🙏

@AllenFang AllenFang marked this pull request as draft February 27, 2025 10:14
@0xi4o
Copy link
Contributor

0xi4o commented Feb 27, 2025

Ah ok. I will wait for your updates @AllenFang.

@AllenFang
Copy link
Author

hi @0xi4o
I just notice that your conflict fix seems mess up the lock file
it's my previous change: https://github.com/FlowiseAI/Flowise/pull/4061/files#diff-32824c984905bb02bc7ffcef96a77addd1f1602cff71a11fbbfdd7f53ee026bb

But after your fix, it seems change a lots (not sure whether it is cause by indent space?

@AllenFang
Copy link
Author

never mind, I get it back and fix the logger part. @0xi4o , please take a look. thank you

@AllenFang AllenFang marked this pull request as ready for review March 3, 2025 08:24
@AllenFang AllenFang requested a review from 0xi4o March 7, 2025 05:24
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