search

grofit / dotween-nodecanvas

Node Canvas actions for DoTween library
12 stars 4 forks source link

DOTween Examples script fix #2

Closed Async0x42 closed 9 years ago

Async0x42 commented 9 years ago

Accidentally had the examples in "DOTween Examples\DOTween", as well I updated the parameters for those scripts (forgot to copy the changes over last time)

The official integrations are all using NodeCanvas.Tasks.-Integration- as the format for their namespace, so I changed it to that, however since it's not an official integration I don't know if that's appropriate to do or not, will leave it up to you (I placed it in a separate commit, I'm new to forks so I don't know if you can select specific commits in a pull request to merge)

P.S. The example scene seems totally busted, I had never opened it prior to updating, so I'm not sure if it's related to updating everything to the latest NodeCanvas

grofit commented 9 years ago

Originally it was Examples/ for the 3rd party ones as that way you would not have loads of different files at a top level it would be nested below, but again if that is what the official mods do then just follow that.

As far as the example scene there was never meant to be one, I just knocked something together quickly and it was just a simple tween along a line. Feel free to remove them and completely replace them with useful examples.

As far as the namespaces go I would just match the folder structure (ignoring the root Assets bit), as long as they are in a namescape they all have custom categories and names to allow them to be included easily.