Am I using this RemoteEvent correctly?

This script is a colouring script, the client is inside a Gui button that can be selected, named after the appropriate colour. The server script is inside ServerScriptStorage. This script is not working in test mode OR in game, I have a part selected and named 'Target' which is a string value, this is the part that needs to be coloured.

This is the client side script;

local player = game.Players.LocalPlayer
local data = script.Parent
local event = game.ReplicatedStorage:WaitForChild("ColouringEvent")
script.Parent.MouseButton1Click:Connect(function()
	print('ok')
	event:FireServer(data.Name)
end)

This is the Server side script;

local event = game.ReplicatedStorage:WaitForChild("ColouringEvent")
event.OnServerEvent:Connect(function(player, colour)
	local Target = game.ReplicatedStorage.Target
	local character = player.CharacterAdded:wait()
	local Get = character:GetChildren()
	for i = 1, #Get do
		if Get[i].ClassName == "Model" then
			local Gget = Get[i]:GetChildren()
			for i = 1, #Gget do
				if Gget[i].Name == Target.Value then
				Gget[i].BrickColor = BrickColor.new(colour)
				print('Check if Coloured')
				end
			end
		end
	end
end)