My text button that awards a badge won't work?

I am trying to make a "badge walk" style game, except instead of walking for a long amount of time, you just click a TextButton. However, my code doesn't work, and nothing gets written into the console.

local button = script.Parent
local BadgeService = game:GetService("BadgeService") -- this gets the service used for badges
local PlayedBadge = 2124637741 -- replace the numbers with your badge id found after you create it
local function onButtonActivated(player)
	print("this should work")
	-- 1
	BadgeService:AwardBadge(player.UserId, PlayedBadge)
end

button.Activated:Connect(onButtonActivated)

I am using a regular script. I have followed many tutorials, but none seem to work.

Update: Because of one of the answers, I am now using this script.

local button = script.Parent
local BadgeService = game:GetService("BadgeService") -- this gets the service used for badges
local PlayedBadge = 2124637741 -- replace the numbers with your badge id found after you create it
local function onButtonActivated(player)
	print("this should work")
	-- 1
	BadgeService:AwardBadge(player.UserId, PlayedBadge)
end

--// mouse click!
button.MouseButton1Click:Connect(onButtonActivated)
 
--// touch
button.TouchTap:Connect(onButtonActivated)

The TextButton works, but now I get an error: 19:15:57.265 - Players.efan343.PlayerGui.SurfaceGui.TextButton.Script:7: attempt to index nil with 'UserId' Also, because apparently, it matters for some reason, it is a script inside a TextButton inside a SurfaceGui inside StartedGui.

You have the wrong event "Activated" event on a textbutton only fires when you set the property of the button Active = true (Guestimate!)

The event you want is called MouseButton1Click or for touch sensitive devices TouchTap

I would change your script to the following:

local button = script.Parent
local BadgeService = game:GetService("BadgeService") -- this gets the service used for badges
local PlayedBadge = 2124637741 -- replace the numbers with your badge id found after you create it
local function onButtonActivated(player)
    print("this should work")
    -- 1
    BadgeService:AwardBadge(player.UserId, PlayedBadge)
end

--// mouse click!
button.MouseButton1Click:Connect(onButtonActivated)

--// touch
button.TouchTap:Connect(onButtonActivated)

Hope this helps! :)

you cant earn a badge in studio, im sure you havent tried to publish the update and test it on roblox site, so go ahead and do that

Try it a different way, something like this should work.

local button = script.Parent
local BadgeService = game:GetService("BadgeService")
 
local function onButtonActivated(player, 2124637741) --This is the badge Id
	local success, badgeInfo = pcall(function()
		return BadgeService:GetBadgeInfoAsync(2124637741)
	end)
	if success then --This confirms if the badge can be rewarded.
		if badgeInfo.IsEnabled then
			local awarded, errorMessage = pcall(function()
				BadgeService:AwardBadge(player.UserId, badgeId)
			end)
			if not awarded then --If the badge was not awarded then it should tell you the error.
				warn("There was a badge error:", errorMessage)
			end
		end
	else
		warn("Error while getting badge info")
	end
end

button.MouseButton1Click:Connect(onButtonActivated)

Hope this helps!

Only the server can award a badge. Make sure you are using a remote event to send info to the server to award the badge.

I'm assuming that you are using a local script since you are doing mousebutton1click.

If you aren't using a local script then that's also an issue, mouse click only works on the client.

One last thing: Awarding a badge through client activation may be unsafe in most cases, exploiters can gain access nearly immediately through this (thats because its the client). the best way to award badges is through touch events or simple automation on the server. I don't know the context though so im assuming you are using it in a way were exploiters obtaining the badge wont matter.