done controller logic