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

Fix Configuration.xml #225

Open
ronie-z-wave opened this issue Feb 1, 2017 · 5 comments
Open

Fix Configuration.xml #225

ronie-z-wave opened this issue Feb 1, 2017 · 5 comments
Labels

Comments

@ronie-z-wave
Copy link
Contributor

ronie-z-wave commented Feb 1, 2017

Maybe also related to #216 , #228?

I've played a bit around with that issue and figured the following out:

  • PUT requests 501 from server (not recognized/known) - is OK i think
  • POST/GET will response 404 if conf/Configuration.xml doesn't exist
    as I know this file should be added by E-UI right? But untill POST isn't able to add this file it isn't possible in that way ...
  • If this file exists (added it manually) GET is working well but POST with changed xml won't change the current one. Request is 200 OK with the unchanged Configuration.xml

checked it with postman:

configuration-xml_post_404

configuration-xml_bug

@PoltoS: for me it looks more like a webserver/modhttp problem ...

@PoltoS
Copy link
Member

PoltoS commented Apr 14, 2017

@zwave-mke I propose to replace this Configuration.xml by a special call to ZWaveAPI (tbd in ZWave/index.js) that will save the file in storage as "ExpertUI-Configuration-" (with zway name!!!). Another call will be used to retrive it from the storage.

It will be JSON and not Configuration.xml.

Agree?

@vades
Copy link
Contributor

vades commented Apr 18, 2017

I agree wit that. Some parts of the UI (configuration, assoc ..) will need to be refactored but JSON is better and faster as XML.

@Kosta-Github
Copy link

so, is there a fix available now?

@michaelhq
Copy link

michaelhq commented Nov 23, 2017

same here with 2.3.6, cannot save parameter with fibraro motion sensor
It would be nice to have a solution...

@vades
Copy link
Contributor

vades commented Nov 23, 2017

There is nothing new... :-(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants