How do you make a quest that if you complete it, you can pick up a keycard?

I've had some troubles with trying to make it so if you complete a quest you can pick up a keycard Idk what the script that is broke, so I'm not advertising, but I'll send you a link of the quest []https://www.roblox.com/library/3704323126/Quest-end[]

If anyone can help me I would be very happy!

local Keycard = "[Keycard Path]"

Keycard.Touched:Connect(function(hit) --Runs when keycard is touched
	local Player = game.Players:GetPlayerFromCharacter(hit.Parent) --Tries to get the player
	if player then --Runs if it was hit by the player
		if player.Quest.Value == true then --Checks to see if a value named quest is equal to true
			local clone = Keycard:Clone() --Clones Keycard
			Keycard.Parent = player.Backpack --Puts keycard in backpack
		end
	end
)