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

Fixed for Unity 2017.1.0f3 and 2017.2.0f3 #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

marcin-jozwikowski
Copy link

I've found your plugin couple weeks ago and realy wanted to use it in newest Unity editor.
Finally I've found some free time to fix it and here it is!

It all comes down to moving whole contructor code to OnEnable function since it is now forbidden to call Application.[anything] in contructor. Removing readonly property from variables was therefore a must.

Hope this would help you get this plugin back to UnityAssetStore.

BTW: In my master branch there are also changes made in CandyFace fork of your repo. AFAIK they are mostly visual but I must say that it looks quite nice :)

@pedro15
Copy link

pedro15 commented Nov 26, 2017

Omg Thank you !!

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