laurent-martin / ets-to-homeassistant

script to convert ETS5 project file to a YAML configuration file suitable for Home Assistant
Apache License 2.0
53 stars 12 forks source link

few suggestions/elements to check #23

Open TocsinChti opened 5 months ago

TocsinChti commented 5 months ago

Hello Gentlemen,

first of all thanks @laurent-martin for the dvpt ! Great idea ; thanks also to other participant.

a few suggestions on the readme & maybe a bug/limitation to be corrected (please note I am a newby in dvpt and therefore my comments are also aimed to permit low level users like me to use your tool ;-) ) :

1) May I recommand adding in your "Important note" that project should not have any password (as highlighted later in the "internal note" but not sure many go reading up to there before testing.

2)Maybe also explaining for the very newby users like me that programm is searching for the file in the directory currently listed in terminal windows before the command line is entered (Yes I am very low leverl user, but I assume representative to part of potential users... just to highlight, I have been looking for 30min-1h & was about to publish here to ask... yes, newby user)

3) from what I tested, initialy my knxproj file had a space in its name... and this did not work (due to syntax command ?); I changed to small letters + no space and it did work ; mabe advise this in important note ? or possibility to support spaces ?