another thing that I just thought of, you are using a group field with sub fields… You can’t update the sub field directly using update_field(). You either have to update the group field with an array of sub fields (field key => value pairs), or you need to use update_sub_field(), https://www.advancedcustomfields.com/resources/update_sub_field/
Yes, a taxonomy field holds and array of term ID as string values.
Welcome to the Advanced Custom Fields community forum.
Browse through ideas, snippets of code, questions and answers between fellow ACF users