search

drpout / boilr

Price alarms for Bitcoin, cryptocurrencies, cryptoassets, futures and options.
http://boilr.mobi
GNU General Public License v3.0
128 stars 67 forks source link

change alarm type greyed out #104

Open algazarra opened 9 years ago

algazarra commented 9 years ago

screenshot_2015-03-13-00-32-34

after creating an alarm, if we go to the alarm's settings, I'm not able to change the alarm type.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

dllud commented 9 years ago

Sure that's on purpose. Besides being technically difficult to change alarm type once created, it also would be a strange and little used use case. You would only keep Exchange and Pair and have to set all the rest.

PS. You agreed to this on the beginning.

algazarra commented 9 years ago

yes I remember, but I don't remember why it had to behave like that. Was there a technical reason?

dllud commented 9 years ago

As I said during that discussion it is doable but too much work for little use. Changing alarm type requires deletion of current alarm, creation of a new one and replacement of the DB (preferably using the same ID).

We might implement this one day if people start coming here by the dozen and requesting this.

algazarra commented 9 years ago

you mean, too much work coding? I'd leave this open. having the option greyed out doesn't look very good and isn't very logical since you can change the alarm type multiple times when creating it. if you can change other parameters, why would this be the only one greyed out, you know?