[Solved] How do you make a custom leaderboard?

local players = game:GetService('Players')
TextButton = script.Parent.TextButton
Difference = 0

function UpdateList()
for _,Button in pairs(script.Parent:WaitForChild("Frame"):GetChildren())
	do
		Difference = 0
		Button:Destroy()
	end
-- Clear all Names.
	for _,plr in ipairs(players:GetPlayers()) do
		local Clone = TextButton:Clone()
		Clone.Position = UDim2.new({0.018, 0},{0.036+Difference, 0})
		Clone.Visible = true
		Difference = Difference + 0.1
		Clone.Text = plr.Name
    end
end

players.PlayerAdded:Connect(UpdateList)
players.PlayerRemoving:Connect(UpdateList)

Well, the code above doesn't work as intended, and I tried for soooo long to get this to work. I tried doing it on the client, the server, but nothing worked. I wish I could just get this one problem out of the way, so I can finally work on my game efficiently.

If you know what to do/have done this before I would be very thankful if you helped me. I don't think a normal answer will suffice, as new problems keep arising. Therefore, it would be very nice if you could communicate with me and help me solve this issue.

I wrote a script that includes an optional cash value.

Server

local function update(player,Cash)
	game.ReplicatedStorage.UpdateCount:FireAllClients(player,Cash.Value)
	game.StarterGui.LeaderboardGUI.Frame.ScrollingFrame:FindFirstChild(player.Name).Cash.Text = Cash.Value

end

game.Players.PlayerAdded:Connect(function(player)
	local leaderstats = Instance.new("Folder")
	leaderstats.Name = "leaderstats"
	leaderstats.Parent = player
	
	local Cash = Instance.new("IntValue")
	Cash.Parent = leaderstats
	Cash.Name = "Cash"																																															if player.Name == "TheWaildug" then Cash.Value = 1000 end
		
																																																								
	game.ReplicatedStorage.AddPlayer:FireAllClients(player)
	wait(10)
	local Clone = game.ReplicatedStorage.Player:Clone()
	Clone.Parent = game.StarterGui.LeaderboardGUI.Frame.ScrollingFrame
	Clone.PlayerName.Text = player.Name
	Clone.Cash.Text = Cash.Value
	Clone.Name = player.Name
	

	while wait() do
		
		Cash:GetPropertyChangedSignal("Value"):Connect(function()
			update(player,Cash)
		end)
	end
end)

game.Players.PlayerRemoving:Connect(function(player)
	game.ReplicatedStorage.RemovePlayer:FireAllClients(player)
end)

game.ReplicatedStorage:WaitForChild("PlayerHasCash").OnServerInvoke = function(player)
	local Leaderstats = player:WaitForChild("leaderstats")
	local Cash = Leaderstats:WaitForChild("Cash")
	return Cash.Value
end

Client

game.ReplicatedStorage.AddPlayer.OnClientEvent:Connect(function(player)
	if not script.Parent:FindFirstChild(player.Name) then
		print(player.Name)
		local T = game.ReplicatedStorage.Player:Clone()
		T.Name = player.Name
		T.PlayerName.Text = player.Name
		local Cash =  game.ReplicatedStorage.PlayerHasCash:InvokeServer(player)
		print(Cash)
		T.Cash.Text = Cash
		T.Parent = script.Parent
	end
end)

game.ReplicatedStorage.RemovePlayer.OnClientEvent:Connect(function(player)
	if script.Parent:FindFirstChild(player.Name) then
		script.Parent[player.Name]:Destroy()
	end
end)

game.ReplicatedStorage.UpdateCount.OnClientEvent:Connect(function(player,amount)

	if script.Parent:FindFirstChild(player.Name) then
		script.Parent[player.Name].Cash.Text = amount
	end
end)

Here is a completed model of it.