You need to check the value that your passing to implode. It should be an array of image IDs. The code looks right. The shortcode must have the list of IDs or is just shows all images attached to the post.
Welcome to the Advanced Custom Fields community forum.
Browse through ideas, snippets of code, questions and answers between fellow ACF users