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

Switching rule from SetEffect to PulseEffect doesn't preserve previous property state #1247

Open
benfrancis opened this issue Aug 2, 2018 · 1 comment

Comments

@benfrancis
Copy link
Member

STR:

  • Add virtual on/off switch
  • Add virtual on/off light
  • Add virtual on/off color light
  • Create a rule that says "If virtual on/off switch is on and virtual on/off light is on, turn virtual on/off color light on"
  • Navigate to things screen
  • Trigger rule by turning on/off switch and on/off light on
  • Navigate to Rules screen and modify rule from "if" to "while"
  • Turn virtual on/off light off

Expected:

  • Virtual on/off color light turns off

Actual:

  • Virtual on/off color light stays lit until it is manually turned off, then the while rule starts working.
@benfrancis benfrancis added the bug label Aug 2, 2018
@benfrancis benfrancis added this to the 0.5.1 milestone Aug 7, 2018
@hobinjk hobinjk self-assigned this Aug 20, 2018
@hobinjk
Copy link
Contributor

hobinjk commented Aug 20, 2018

Looking into this the problem is that the rule completely resets its state whenever modified. For example, setting up "While door is open turn light to red", opening the door, modifying the color to blue, then closing the door will not reset the color to its original state.

I started working on a fix but it's proving a bit hard to work out.

@benfrancis benfrancis changed the title Switching if/else doesn't immediately take effect Switching if/while doesn't immediately take effect Aug 21, 2018
@benfrancis benfrancis modified the milestones: 0.5.1, 0.6 Aug 21, 2018
@benfrancis benfrancis modified the milestones: 0.6, 0.7 Oct 23, 2018
@mrstegeman mrstegeman removed this from the 0.7 milestone Jan 9, 2019
@hobinjk hobinjk changed the title Switching if/while doesn't immediately take effect Switching rule from SetEffect to PulseEffect doesn't preserve previous property state Feb 5, 2019
@mrstegeman mrstegeman added backlog and removed next labels Feb 13, 2019
@mrstegeman mrstegeman removed the backlog label Apr 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants