almost working i think - loop todo next