Closed nailend closed 1 month ago
Hello @nailend! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
Only black and coveralls fails, as in all the PRs.
@jokochems could you review the adaptions in this PR please?
Currently the invest relations can not change in between the periods of a multi-period investment. I dont see a reason why this shouldnt be possible, so I just change the few necessary parts.
@jokochems was there a specific reason why you did not adapt this as well or did you just not need it?