add todo for ok set