fix to use ok()