did i do this right