Follow-up from "Populate AI Catalog Version Created By"

The following discussion from !214899 should be addressed:

  • @ashmckenzie started a discussion:

    question / suggestion: Is there any disadvantage to perform the merging within build_new_version(), e.g.

    def build_new_version(version_params)
      item.build_new_version(version_params).merge(created_by: current_user)
    end

    and then (back to :sweat_smile::

              build_new_version(version_params)
Edited by 🤖 GitLab Bot 🤖