search

ColeBennett / binance-auto-blacklist

Automatically disables trading for new Binance listings in ProfitTrailer
34 stars 6 forks source link

App removes // Comments from appsettings.json #14

Open smoochy opened 6 years ago

smoochy commented 6 years ago

Hi there,

first of all: nice little tool which can save a lot of money :)

I just found a glitch:

I like to have the comments like

// All_max_buy_spread

behind my values in the appsettings.json of PTFeeder.

After having run your tool, those comments are gone. I've added some comments again, closed your tool, ran it again and again all comments had been deleted.

It would be very helpful, of comments would not get deleted when your app runs.

Thank you.

Windows Server 2016 on a VPS, Java 8 Update 162. App startet via .bat file.

lincolnaleixo commented 6 years ago

that would be nice. I can't use this tool now because of that, I rely a lot on my comments.

HeebyGeeby commented 6 years ago

In fairness to the developer, comments aren't allowed for in the json RFC. I know both PT and Feeder devs use them, as do I, but I can see how an app might legitimately remove them.