Fix delete button logic

[CarlosABVA]

Fixes #958.

About the edit note on the issue, I haven´t been able to replicate an account with splits that could be deleted

[argaen]

I haven´t been able to replicate an account with splits that could be deleted

Actually.... I just discovered why you can't reproduce. So the problem here is the partial data loaded during the period. As in, if you select a period of three months, we only load the splits for those three months for that account so the condition of splits.length === 0 will be true. You can try it, I was able to reproduce it.

However, your code already improves current behavior so I'm happy to merge as long as we add a test and fix the pipeline! If you need a hand, happy to help.

[CarlosABVA]

However, your code already improves current behavior so I'm happy to merge as long as we add a test and fix the pipeline! If you need a hand, happy to help.

Alrite I added a couple tests to cover the updated logic. About the failing pipeline, SonarCloud is complaining about a missing token, not sure where this comes from

[argaen]

looks good, merging!

ignoring the scanner error since tests + lint passed