RoboZonky / natural-strategy-setup

Webový konfigurátor investiční strategie pro RoboZonky
https://robozonky.github.io/konfigurace-strategie/
2 stars 1 forks source link

Prepare for upcoming Zonky changes #81

Closed triceo closed 3 years ago

triceo commented 3 years ago

Zonky will soon roll out the following changes:

I am making the necessary changes. I was not able to find any references to the original 700k amount, but I believe I changed all the term stuff.

Nice clean code, was easy to modify! Please point out if I skipped anything.

jhrcek commented 3 years ago

@triceo Let me know when you want this deployed and I'll create a PR to robozonky.github.io

triceo commented 3 years ago

@jhrcek Please create the PR and I'll merge it when I need to. I expect to be releasing 6.3.3 later today.

jhrcek commented 3 years ago

Done. And regarding

I was not able to find any references to the original 700k amount

The tool doesn't enforce any max limit there. Should it or is it ok as is? Screenshot from 2021-02-28 10-50-17

triceo commented 3 years ago

I think it would be reasonable to have some limit there. Since there is no way how the amount would ever exceed 900k, let's set that as the limit.

That said, I will not be adding a limit to the strategy itself.

jhrcek commented 3 years ago

Of course, I'll add it later