That looks good and would be a nice UI improvement if replacing "Completed" with "Failed" is too complicated.
I would prefer to replace "Completed" with "Failed" if a sync is not successfully finished.
My open issue to this topic: https://github.com/laurent22/joplin/issues/3851#issuecomment-705723342