genielabs / HomeGenie

HomeGenie, the programmable automation intelligence
https://homegenie.it
GNU General Public License v3.0
392 stars 155 forks source link

Can build and debug but having javascript issues. #49

Closed jarrettv closed 9 years ago

jarrettv commented 9 years ago

I've been looking to work on homegenie and fix my zwave thermostat but I'm having front-end issues.

If you clone the repo, open in visual studo, and run HomeGenie, the app runs fine but you'll see what looks to be javascript issues.

I've dug around and perhaps it is an issue with msbuild. I noticed the UI csproj file appears "dirty" in that there may be some merging errors in there.

Gene, any advice? Is the setup/install process doing something the build isn't? Screenshot below:

image

jarrettv commented 9 years ago

More investigation image

jarrettv commented 9 years ago

I've also tried opening in monodevelop (xamarinstudio) and it has issues running solution. Gene, have you tried opening the solution from a clean environment lately?

genemars commented 9 years ago

Could you try with source code from sourceforge repo? There might be some missing file in the github repo. Did not try myself a clean github checkout since long and under Windows since ever longer time.

genemars commented 9 years ago

One advice. Did you try running Visual Studio/Xamarin as Administrator?

genemars commented 9 years ago

Update: just tried a clean checkout from github under Linux and everything is ok. I'll try Windows later. Let you know.

genemars commented 9 years ago

made some fixes, checkout latest code and let me know if it's working now.

genemars commented 9 years ago

Had a chance to verify building in windows and fixed errors. Also updated building/packaging instructions https://github.com/genielabs/HomeGenie (see README.md displayed at the bottom of the page).

jarrettv commented 9 years ago

Good news. You've fixed the problems and a mighty good decision on removing the UI project.

I did have to change the target framework from .NET 4.0 to .NET 4.5 due to the Jint library. Otherwise visual studio couldn't resolve the reference.

Now that the UI is behaving, I can finally dig into my zwave thermostat issues. Thanks!

genemars commented 9 years ago

Please checkout the very latest source code. It should also fix Jint error, so no need to change target framework. HomeGenieUI was a very very old work-around for an older Monodevelop version, but this is not needed anymore. Things now are more clean. Let me know if you encounter any other issue. Thanks!