search

ransome1 / sleek

todo.txt manager for Linux, Windows and MacOS, free and open-source (FOSS)
https://github.com/ransome1/sleek/wiki
MIT License
1.34k stars 104 forks source link

Fix for issue #429 #433

Closed zerodat closed 1 year ago

zerodat commented 1 year ago

This fixes issue #429 "Delete doesn't work".

The bug was introduced in 1.2.7 commit 2877852bd7580c6eff141bdba6ad9bdfba2d52e7 in which a parameter "index" was added to a new deleteTodo function, but the callers were not changed to add the index to the calls. This PR just adds the parameter to the two calls.

ransome1 commented 1 year ago

@zerodat thank you so much for your fast response! I merged and tested it. It will part of the next release. In general I should take a stronger focus on automated testing. Something like this should not have happened :/