Rooms Manager: Smarter Rooms: Personalized home automation with Occupancy

rooms
smartrules
microlocation
project_sensors
manager

#908

both of these values appear to be null for some reason … but they both have defaultvalue so should not be null … wondering if that is the issue:

are you comfortable editing the rooms manager app in the IDE directly if i give you the lines to replace?


#909

Sure, just walk me through it


#910

cool. open rooms manager app in the IDE and replace these lines:

section("Announce only between hours:")     {
    if (playerDevice)        {
        input "startHH", "number", title: "From hour?", description: "2 digit hour in 24 hour format", required: true, multiple: false, defaultValue: 7, range: "1..${(endHH ?: 23)}", submitOnChange: true
        input "endHH", "number", title: "To hour?", description: "2 digit hour in 24 hour format", required: true, multiple: false, defaultValue: 23, range: "${(startHH ?: 23)}..23", submitOnChange: true
    }
    else        {
        paragraph "Announce from hour?\nselect either presence or time announcement to set"
        paragraph "Announce to hour?\nselect either presence or time announcement to set"
    }
}

with these lines:

section("Announce only between hours:")     {
    if (playerDevice)        {
        input "startHH", "number", title: "From hour?", description: "2 digit hour in 24 hour format", required: true
        input "endHH", "number", title: "To hour?", description: "2 digit hour in 24 hour format", required: true
    }
    else        {
        paragraph "Announce from hour?\nselect either presence or time announcement to set"
        paragraph "Announce to hour?\nselect either presence or time announcement to set"
    }
}

save and publish the app then open the rooms manager app … lets see if the same error shows up.


#911

Error message went away


#912

awesome. put in 7 and 23 respectively and save it.

then reverse the line replacement above. then open the rooms manager app … does the error come back?


#913

7/23 entered then reversed and save it. The error did not come back


#914

enjoy! :slight_smile:


#915

Thanks a bunch. Quick question. If I want the announcements available at all times can I change the 7/23 to 7/7 or 12/12 something like that?


#916

you are welcome.

try 0 / 23 … if it does not work i am always here. :slight_smile:


#917

GOT IT! Nice work


#918

update from github before you use 0 / 23 :wink:


#919

CRAP! I changed it to 0/23 and the error came back


#920

it allowed you to save it with 0 / 23?

update from github and try it before trying the previous code replacement.


#921

Went I hit save, it I think it allowed me to save it (cant remember). But I backed out then tried to go back in and I got the error message


#922

yeah backing out isnt generally a good idea in ST :slight_smile:

update from github and try it before trying the previous code replacement.


#923

ok. I updated from GitHub. About to do the code replacement. Be right back


#924

wait dont do the code replacement yet.

first try with new code from github.


#925

Sorry, just did it.


#926

just update from github again and try it.


#927

Ok…updated tried again. No error message. Now I am so afraid to touch anything. Don’t wont to break it again…lol