search

oukene / smartthings_customize

10 stars 7 forks source link

async_set_hvac_mode 수정 #16

Closed clipman closed 1 year ago

clipman commented 1 year ago

hvac 모드 변경 동작 확인했습니다.

clipman commented 1 year ago

main component에서는 그렇게 되는데 sub component에서는 모드 변경과 동시에 전원이 안켜집니다. 그래서 sub component에서는 전원이 꺼져 있을 경우에 모드 변경은 무시해야 되네요. 아니면 시간 간격을 좀 두고 명령을 호출할 수 있다면 가능할 수도 있겠네요.

oukene commented 1 year ago

main component에서는 그렇게 되는데 sub component에서는 모드 변경과 동시에 전원이 안켜집니다. 그래서 sub component에서는 전원이 꺼져 있을 경우에 모드 변경은 무시해야 되네요. 아니면 시간 간격을 좀 두고 명령을 호출할 수 있다면 가능할 수도 있겠네요.

아 그게 또 컴포넌트에 따라 다르구만요... pr 보내주신거에 mode 체크만 한번 추가하면 될것 같아요

clipman commented 1 year ago

main component에서도 제 기기에선 switch on을 날려야 하네요.

oukene commented 1 year ago

main component에서도 제 기기에선 switch on을 날려야 하네요.

그럼 나중에 추가된 main 에서만 switch on 처리 하는거 빼버리고 ( 처음 수정버전 ) mode 가 동일할 경우에는 모드 커맨드는 스킵, 모드가 다를 경우에만 모드 설정 커맨드 날리도록 수정하면 완벽할것 같네요

clipman commented 1 year ago

switch on은 반드시 필요합니다. 모드 변경을 비교하도록 수정했습니다. 현재 sub component에서는 전원이 꺼져 있을 경우에 전원만 켜지고 모드 변경은 이루어 지지 않네요.

oukene commented 1 year ago

main component에서도 제 기기에선 switch on을 날려야 하네요.

삼성 에어컨 다 그럴것 같네요, 저는 결국 ir신호를 쏘는거라 그렇진 않았는데 삼성에어컨 전부 다 그럴듯 하네요

oukene commented 1 year ago

switch on은 반드시 필요합니다. 모드 변경을 비교하도록 수정했습니다. 현재 sub component에서는 전원이 꺼져 있을 경우에 전원만 켜지고 모드 변경은 이루어 지지 않네요.

확인했어요~ merge 하고 조건문 밖으로 빼둘께요