Why doesn't this remote event fire properly?

So I'm making a remote event which when fires changes a labels text to the player's name, but it's not working there are no error it just doesn't work.

here is the remote event script :

replstor = game:GetService("ReplicatedStorage")
rmtfol = replstor:WaitForChild("Remotes")
rmt = rmtfol:WaitForChild("notadmin")
startgui = game:GetService("StarterGui")
scrngui = startgui:WaitForChild("ScreenGui")
notification = scrngui:WaitForChild("notification")
txtlbl = notification:WaitForChild("notilabl")
groupid = 3205396

rmt.OnServerEvent:Connect(function(ply)
	
	for i,v in pairs(game:GetService("Players"):GetPlayers()) do	
	
		if v:GetRankInGroup(groupid) >= 6 then	

			-- txtlbl:TweenPosition(UDim2.new(0, 0,0, 760), 'Out', 'Bounce')
			txtlbl.Visible = true
			txtlbl.Text = ply.Name .. " hi"
			wait(3)
			txtlbl.Text = "nil"
			txtlbl.Visible = false
			-- txtlbl:TweenPosition(UDim2.new(0, 0,1.7, 760), 'Out', 'Bounce')
		
		end
		
	end
	
end)

and here is the script that fires it:

function onClicked(ply)

		game:GetService("ReplicatedStorage"):WaitForChild("Remotes"):WaitForChild("notadmin"):FireServer(ply.Name)

end
 
script.Parent.ClickDetector.MouseClick:connect(onClicked)