daikin-control icon indicating copy to clipboard operation
daikin-control copied to clipboard
verified publisher icon ael-code

Master

Open stagadet opened this issue 9 years ago • 5 comments

Added test configuration B Added possibility for configuration B to use both GET and POST http requests for all commands Added description of commands for:

  1. Power consumption
  2. Special modes
  3. WiFi LEDs
  4. Timers

stagadet avatar Dec 11 '16 17:12 stagadet

One thought on the GET/POST topic: Even if Daikin implemented something that breaks REST "ideas" (e.g. using GET calls to change values) the question is if this should be used ... It's better to follow common standards for usage, or ?!

Apollon77 avatar Feb 27 '17 11:02 Apollon77

I don't know if it is better to use GET or POST to modify parameters in this case. The official Daikin app that I tested uses GET, while I think ael-code's GUI uses POST. In my case they both work, maybe also in older setups. In any case I think it is better to show both possibilities in the 'documentation'.

stagadet avatar Feb 28 '17 20:02 stagadet

@ael-code I have an enhanced version of this pull request in my fork at https://github.com/Apollon77/daikin-control in an new file (just to be able to get updates from upstream without all those conflict stuff) I also used your infos to implement an nodejs library (https://github.com/Apollon77/daikin-controller) and verified nearly all of those new details with this lib and my two daikin devices.

Maybe you can check my README_updated.md nd tell me what you see on change needs in preparation to make an official pull request ... Or should I just send it? Would be great. Thank you for all you great work here

Apollon77 avatar Jun 21 '17 09:06 Apollon77

Why this information still don't added in readme file? It's quite useful.

WildRat avatar Jan 06 '18 20:01 WildRat

Please merge this PR, because I accidentally found the missing endpoints

andras-tim avatar Feb 20 '20 13:02 andras-tim