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]Fail to yarn build #1153

Closed
nobody4t opened this issue Oct 30, 2023 · 7 comments
Closed

[BUG]Fail to yarn build #1153

nobody4t opened this issue Oct 30, 2023 · 7 comments

Comments

@nobody4t
Copy link

Describe the bug

flowise-components:build: nodes/vectorstores/Milvus/Milvus_Upsert.ts(255,59): error TS2551: Property 'IndexNotExist' does not exist on type 'typeof ErrorCode'. Did you mean 'INDEX_NOT_EXIST'?

To Reproduce
Steps to reproduce the behavior:

change to main branch, pull and yarn build

Expected behavior
Building should be OK

This commit 60ac084 produce this.

@neo4dev
Copy link

neo4dev commented Nov 1, 2023

same issue deploying on render.com
Nov 1 08:53:48 PM #17 10.70 flowise-ui:build:
Nov 1 08:54:21 PM #17 44.39 flowise-components:build: nodes/vectorstores/Milvus/Milvus_Upsert.ts(255,59): error TS2551: Property 'INDEX_NOT_EXIST' does not exist on type 'typeof ErrorCode'. Did you mean 'IndexNotExist'?
Nov 1 08:54:21 PM #17 44.57 flowise-components:build: error Command failed with exit code 2.
Nov 1 08:54:21 PM #17 44.57 flowise-components:build: info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Nov 1 08:54:21 PM #17 44.60 flowise-components:build: ERROR: command finished with error: command (/usr/src/packages/packages/components) yarn run build exited (2)

@nobody4t
Copy link
Author

nobody4t commented Nov 1, 2023

really? It should be INDEX_NOT_EXIST instead of IndexNotExist. See below:
Screenshot 2023-11-01 at 9 20 47 PM

@nobody4t
Copy link
Author

nobody4t commented Nov 2, 2023

@HenryHengZJ can you verify this issue. Just revert the commit would resolve it.

@HenryHengZJ
Copy link
Contributor

this is strange, a while back it was the other way round - #1136

@nobody4t
Copy link
Author

nobody4t commented Nov 3, 2023

this is strange, a while back it was the other way round - #1136

The right reference should be ErrorCode.INDEX_NOT_EXIST or use "IndexNotExist" directly, not just ErrorCode.IndexNotExist.

@neo4dev
Copy link

neo4dev commented Nov 3, 2023

this is strange, a while back it was the other way round - #1136

I sync the latest repo and it works now. Thanks.

@nobody4t
Copy link
Author

nobody4t commented Nov 7, 2023

Oh the error code is fron milvus not flowise.

@nobody4t nobody4t closed this as completed Nov 7, 2023
JohnBQuinn pushed a commit to JohnBQuinn/Flowise that referenced this issue Jun 7, 2024
Co-authored-by: UUUUnotfound <31206589+UUUUnotfound@users.noreply.github.com>
Co-authored-by: Hexiao Zhang <731931282qq@gmail.com>
Co-authored-by: heheer <71265218+newfish-cmyk@users.noreply.github.com>
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

No branches or pull requests

3 participants