scriptable-events icon indicating copy to clipboard operation
scriptable-events copied to clipboard

Do you intend to add Unity Visual Scripting support?

Open Emilian5 opened this issue 3 years ago • 7 comments

Hello.

I was preparing to refactor my own SO events system implementation, and during preparation I've look to see if someone shared their similar development. So here I am. Your implementation is more, let's say evolved. Good job, and thanks for sharing.

During the development of my game, I found that it was hard to follow the Event Listeners on GO, especially when multiple Events Listeners were added besides other MonoBehaviours. The solution (for me) was Unity Visual Scripting. I moved all the Event's to Unity Visual Scripting graphs, and now they are easier to follow, to trigger, etc., especially from the point of view of a designer.

Do you have any plans to add support for Unity Visual Scripting? Or, if you receive a pull request with Unity Visual Scripting support will you consider accepting it?

Thank you.

Emilian5 avatar Aug 12 '22 13:08 Emilian5

Hey, thank for taking interest in this package!

In regards to visual scripting (Bolt) I personally don't have that much experience and haven't used it at all. So most likely I wont investigate this personally.

However, I think this would be a good addition to the package depending on how this is done. If lets say integration would require a change in every event script I'd be a bit reluctant to add it as this would greatly increase maintenance. If its just a few scripts with some compiler directives (maybe there is a better way, unsure) I think that'd be fine.

Generally speaking the core idea of this package is staying simple to keep the maintenance as low as possible. However, I think integrations with other packages/tools is fine as that doesn't change the core functionality in major ways and moves the responsibility to other parties.

Anyway, if you'd like to take a shot at this go ahead!

Edvinas01 avatar Aug 12 '22 14:08 Edvinas01

None of your existing code will be modified. UVS nodes will be wrappers around current code.

Will allocate more time to better investigate, and will let you know.

Thank you.

Emilian5 avatar Aug 12 '22 14:08 Emilian5

Closing as I'm not planning to add this feature unless someone else decides to contribute - will re-open then.

Edvinas01 avatar Jan 11 '24 10:01 Edvinas01