Is there anything wrong with this script?

I believe I have it set up correctly, but I don't know if I should use 'plr' or 'user' because the player is already defined as 'plr'?

local PointsService = game:GetService("PointsService")
function onJoin(plr)
	wait(3600)
	Game:GetService("BadgeService"):AwardBadge(plr.userId,31678972)
			PointsService:AwardPoints(plr.userId, 7)

end
game.Players.PlayerAdded:connect(onJoin)

Sadly, as I own the badge and can't get it, i can't test this out unless i want to stay in the game for 1 hour.

Seems right. Just to be sure I would put the time for 10 seconds and test it on an alt.

wait(60*60) --ocd sry