Is this right way to make player teleport?

script.Parent.Touched:Connect(function(hit)
	if hit.Parent:FindFirstChild("Humanoid") then
		hit.Parent:MoveTo(script.Parent.Parent.Part.Position)
	end
end)

I have seen people use HumanoidRootPart and CFrame for teleport but is there anything wrong with making player teleport this way? it seems to work tho.

You can use PivotTo.

character:PivotTo(part.CFrame)