RemoteEvent only firing once

Help and Feedback Scripting Support
,
#1

So I have been making a script where a gui hides until main menu is over and the player enters intermission for a round based system game (The GUI will also hide when the round starts). Well, once someone wins the game and returns to intermission, the remote events I put in the code won’t run again for some reason. It only works when the player first enters intermission after main menu is over and I do not know why this is happening, can someone please help me? It will be great if you are willingly to stop by to help me.

Here’s my part of the code:

mistake
mistake929×303 18.8 KB

if you need anymore code feel free to ask me!

#2

Try removing that if statement where it checks the team, I think its useless because you’re literally assigning the team the line before and maybe something from the ES and SS remotes could be causing the problem (i got this on the top of my i doubt what i said works)

#3

i just tried what you recommended for me to do but still no success.

here are more photos of my code if you need them:

mistake2
picture of the remote event receiving the FireAllClients

mistake3
picture of RemoteEvent sending message to all clients to make the gui invisble

#4

can you send/ copy paste the text on the image please? i want to try something

#5

okay one second be right back (30 letter)

#6

do you mean all of the code or just the 2 image i just sent you

#7

all, no need to send anything else just that one script

#8 
game.ReplicatedStorage.ChangeTeam.OnServerEvent:Connect(function()
	for i, v in pairs(Players:GetChildren()) do
		if v:IsA("Player") then
			v.Team = game.Teams["Not Playing"]
			ES:FireAllClients()
			SS:FireAllClients()
		end
	end
end)
#9

SS.OnClientEvent:Connect(function()

button.Visible = true

end)

SH.OnClientEvent:Connect(function()

button.Visible = false

end)

#10 
for i, player in pairs(plrs) do 
	player.Team = game.Teams.Playing
	EH:FireAllClients()
	SH:FireAllClients()
#11

You have an unnecessary conditional for checking the team.

Also, that’s gonna fire for all players multiple times, why not use one big server script to do it?

#12

Yeah that was what I was saying

#13

The thing I was trying to do didnt work so I just removed the condition and changed the Children to Players so it doesnt get any other object other than a player

game.ReplicatedStorage.ChangeTeam.OnServerEvent:Connect(function()
	for i, player in pairs(game:GetService("Players"):GetPlayers()) do
		player.Team = game.Teams["Not Playing"]
		ES:FireAllClients()
		SS:FireAllClients()
	end
end)
#14

idk man im a new scripter thats all

#15

Its a loop idk whats wrong it seems fine it shouldnt be firing once

#16

That’s gonna fire all clients multiple times.
Put your events outside the for loop instead:

for i, player in pairs(plrs) do 
	player.Team = game.Teams.Playing
end
EH:FireAllClients()
SH:FireAllClients()
#17

I think he WANTS them to run multiple times?

#18

that seems like a sensible thing to do, be right back I’m checking if it works

#19

idk really, I’m just going to test if it works or not

#20

That would be strange, since that’d probably be the initialization multiple times, or something.