Other than this I don’t see an issue unless ACF is unable to figure out the post ID. Are you using the global $post variable for your loop? If not then you need to supply the correct post ID to get the value from in the second argument to the_field() or get_field().
Welcome to the Advanced Custom Fields community forum.
Browse through ideas, snippets of code, questions and answers between fellow ACF users