how can i fix this teleport taht wont work ?

Sorry to say but almost every single line here is improper. I'll rewrite the script and do my best to explain what happened as best as I can.

script.Parent.Touched:connect(function(hit)
	local humanoid = hit.Parent:FindFirstChild('Humanoid')
	if humanoid then
		hit.Parent.HumanoidRootPart.CFrame = CFrame.new(192.713, 7.374, -898.129)
	end
end)

So the first thing I want to point out is the touched event. You'll want to have this if statement here to make sure the thing touching the part is a player and not another part.

local humanoid = hit.Parent:FindFirstChild('Humanoid')
if humanoid then
	-- Script stuff
end

Also, touched events detect the character, not the player. So instead of doing GetPlayerFromCharacter() (This line here is completely messed up anyways, it would take forever to explain) you can just access the character through the hit.Parent.

Lastly, the HumanoidRootPart is the PrimaryPart for the character. You'll always want to use CFrame and not Vector3 when teleporting the player.

I've tested this script and it works perfectly. Enjoy!