TechnionYP5777 / Smartcity-Smarthouse

Smartcity-Smarthouse
11 stars 5 forks source link

Adding to dashboard crashes #240

Closed DavidCohen55 closed 7 years ago

DavidCohen55 commented 7 years ago

When I've tried to add to this screen I've got an error

image image

inbalzukerman commented 7 years ago

@DavidCohen55 did you add a data-path to listen to?

DavidCohen55 commented 7 years ago

How can I do it? I wasn't able to add sensors...

inbalzukerman commented 7 years ago

By opening the <choose path> combobox. Please pay attention there should be sensors connected to the system before 😄

DavidCohen55 commented 7 years ago

OK, there isn't any sensors on my system, I will try now to work with your new jar and then comeback also isn't not choosing a path leads to a crash should be a bug?

inbalzukerman commented 7 years ago

It is, of course, I'm trying to help you find a workaround.

DavidCohen55 commented 7 years ago

I've successfully added a widget 🎊 But I've found another error, if after you choose Widget Type, but don't press on any of them but fill the other fields the app shows NullPointerException

inbalzukerman commented 7 years ago

@DavidCohen55 you are describing the same type of error- choosing one of the widgets types is necessary for a successful widget configuration.

Thank you for your review, it will be taken into consideration 👍

inbalzukerman commented 7 years ago

Thank you for your review, the bug you described has been fixed 👍