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

Restrict interfaces to be connected only if the same type #436

Open
terion-name opened this issue Nov 23, 2024 · 2 comments
Open

Restrict interfaces to be connected only if the same type #436

terion-name opened this issue Nov 23, 2024 · 2 comments

Comments

@terion-name
Copy link

terion-name commented Nov 23, 2024

First of all, really great library, thank you!
But one thing that I was surprised to understand on half-way of development with it: it doesn't restrict any type-to-type connections. Text to number, custom types, etc. This is very strange and looks like a bug. Or this somehow can be tuned?

UPD
Found that docs say that only same-type connectons are allowed, but in reality – it allows any. Using 2.6.0 version in nuxt 3

@newcat
Copy link
Owner

newcat commented Nov 24, 2024

Unfortunately I am unable to reproduce your issue. Here is what I've tried: https://stackblitz.com/edit/vitejs-vite-c3bey8?file=src%2FApp.vue

Can you verify you're doing the same? Specifically that you have added this: new BaklavaInterfaceTypes(editor, { viewPlugin: baklavaView });

@terion-name
Copy link
Author

Unfortunately I am unable to reproduce your issue. Here is what I've tried: https://stackblitz.com/edit/vitejs-vite-c3bey8?file=src%2FApp.vue

Can you verify you're doing the same? Specifically that you have added this: new BaklavaInterfaceTypes(editor, { viewPlugin: baklavaView });

ah, you need to manually register all types and manually set each input and output socket... it's not obvious from docs and really should be on getting started. thank you

But I don't get, why built-in interfaces do not have types set? For custom ones I can set it at definition and not repeat at each socket at each node, but with built-ins it is not the case

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

2 participants