Just chiming in on another thing. The get_sub_field() function does not need (nor take) a second parameter of the post id so please remove that. You’ll only need the field name as a single parameter in any get_sub_field() or the_sub_field().
Welcome to the Advanced Custom Fields community forum.
Browse through ideas, snippets of code, questions and answers between fellow ACF users