Will this work properly?

What this code should do, is award every player in the server 1 PP every 10 minutes:

local PPS = game:GetService("PointsService")
while PPS:GetAwardablePoints() >= game.Players.NumPlayers do
	for i,v in pairs (game.Players:GetChildren()) do
		PPS:AwardPoints(1,v.userId)
	end
	wait(10*60)
end

while wait(60*10) do
local PPS = game:GetService("PointsService")
if PPS:GetAwardablePoints() >= game.Players.NumPlayers then
for i,v in pairs(game.Players:GetChildren()) do
PPS:AwardPoints(1,v.userId)
end
end
end

It does it every 600 seconds, this works much better as time in my opinion should be used for the while wait() do key and then award after that long. However, if you wanted you could create a countdown timer that is a bit more difficult however it allows for some room of functions under it. But for this way your using I doubt you need that.