RemoteEvent Issues

Help and Feedback Scripting Support
#1

I’m making a SurfaceGui that sits both in the server and inside of the client’s GUI.

But when I test the local server, it pops up for the client that wrote the text in the TextBox and the server but does NOT show any other client.

What am I doing incorrectly?

--ServerScript Snippet
local function updateText(player, middle)
	script.Parent.SurfaceGui.Middle.Text = middle
	print('fired function')
	if player.Pink.Value == true then
		script.Parent.SurfaceGui.Middle.TextColor3 = Color3.fromRGB(255, 152, 220)
	elseif player.DarkBlue.Value == true then
		script.Parent.SurfaceGui.Middle.TextColor3 = Color3.fromRGB(8, 24, 124)
	elseif player.LightBlue.Value == true then
		script.Parent.SurfaceGui.Middle.TextColor3 = Color3.fromRGB(29, 183, 248)
	elseif player.Yellow.Value == true then
		script.Parent.SurfaceGui.Middle.TextColor3 = Color3.fromRGB(255, 255, 0)
	elseif player.Purple.Value == true then
		script.Parent.SurfaceGui.Middle.TextColor3 = Color3.fromRGB(170, 0, 170)
	elseif player.Green.Value == true then
		script.Parent.SurfaceGui.Middle.TextColor3 = Color3.fromRGB(75, 151, 75)
	elseif player.Black.Value == true then
		script.Parent.SurfaceGui.Middle.TextColor3 = Color3.fromRGB(0,0,0)
	end
	ClientToServer:FireAllClients(script.Parent.SurfaceGui.Middle.Text)
end
RemoteEventMiddle.OnServerEvent:Connect(updateText)

--LocalScript Snippet

local function middleUpdate()
	textBox.Middle:GetPropertyChangedSignal("Text"):Connect(function()
		wait(waittime)
		RemoteEvent:FireServer(textBox.Middle.Text)
		if game.Players.LocalPlayer.Pink.Value == true then
			script.Parent.Middle.TextColor3 = Color3.fromRGB(255, 152, 220)
		elseif game.Players.LocalPlayer.DarkBlue.Value == true then
			script.Parent.Middle.TextColor3 = Color3.fromRGB(8, 24, 124)
		elseif game.Players.LocalPlayer.LightBlue.Value == true then
			script.Parent.Middle.TextColor3 = Color3.fromRGB(29, 183, 248)
		elseif game.Players.LocalPlayer.Yellow.Value == true then
			script.Parent.Middle.TextColor3 = Color3.fromRGB(255, 255, 0)
		elseif game.Players.LocalPlayer.Purple.Value == true then
			script.Parent.Middle.TextColor3 = Color3.fromRGB(170, 0, 170)
		elseif game.Players.LocalPlayer.Green.Value == true then
			script.Parent.Middle.TextColor3 = Color3.fromRGB(75, 151, 75)
		elseif game.Players.LocalPlayer.Black.Value == true then
			script.Parent.Middle.TextColor3 = Color3.fromRGB(0,0,0)
		end
	end)	
end

ClientToServer.OnClientEvent:Connect(function(text)
	textBox.Middle.Text = text
end)
#2

Note you should also be filtering user genorated text that is shown to other users.
https://developer.roblox.com/en-us/articles/Text-and-Chat-Filtering

#3

Okay, will do. But the issue is, that it doesn’t show to other clients.

#4

I would add in some more print to find out what is running where as the issue is probably due to your setup rather than the code itself.

Also avoid putting event connection inside of functions as they will connect a new event each time e.g. middleUpdate function.

You also want to avoid spamming the remote event and include a delayed remote event call.