What Phil means is (If I understand it correctly 😉 ) is define something like $post_id right after the while.
$post_id = get_current_ID();
Then use get_sub_field('service_manager_name', $post_id)
instead of get_sub_field('service_manager_name')
.
-=-
Maybe I’m way off, but why is it necessary to setup the post_data ?
$post is not used in the given code and I don’t see any functions which can’t run without this being setup.