Im making a keylock and I can't find out whats wrong?

Its not the best way to do it, but its the way I did it. There's no errors, the only thing that works is the close:

local Keys = script.Parent.Parent.KeysFrame
local zero = Keys.zero
local one = Keys.one
local two = Keys.two
local three = Keys.three
local four = Keys.four
local five = Keys.five
local six = Keys.six
local seven = Keys.seven
local eight = Keys.eight
local nine = Keys.nine
local x = Keys.x
local finish = Keys.finish
local no1Txt = script.Parent.N1.Text
local no2Txt = script.Parent.N2.Text
local no3Txt = script.Parent.N3.Text
local no4Txt = script.Parent.N4.Text
local no1Crct = 1
local no2Crct = 9
local no3Crct = 5
local no4Crct = 3
local no1Correct = false
local no2Correct = false
local no3Correct = false
local no4Correct = false


zero.MouseButton1Click:Connect(function()
	if script.Parent.Value.Value == 1 then
		no1Txt = "0"
	elseif script.Parent.Value.Value == 2 then
		no2Txt = "0"
	elseif script.Parent.Value.Value == 3 then
		no3Txt = "0"
	elseif script.Parent.Value.Value == 4 then
		no4Txt = "0"
	end
	script.Parent.Value.Value = script.Parent.Value.Value + 1
	if script.Parent.Value.Value == 5 then
		script.Parent.Value.Value = 1
	end
	
end)

one.MouseButton1Click:Connect(function()
	if script.Parent.Value.Value == 1 then
		no1Txt = "1"
	elseif script.Parent.Value.Value == 2 then
		no2Txt = "1"
	elseif script.Parent.Value.Value == 3 then
		no3Txt = "1"
	elseif script.Parent.Value.Value == 4 then
		no4Txt = "1"
	end
	script.Parent.Value.Value = script.Parent.Value.Value + 1
	if script.Parent.Value.Value == 5 then
		script.Parent.Value.Value = 1
	end
end)

two.MouseButton1Click:Connect(function()
	if script.Parent.Value.Value == 1 then
		no1Txt = "2"
	elseif script.Parent.Value.Value == 2 then
		no2Txt = "2"
	elseif script.Parent.Value.Value == 3 then
		no3Txt = "2"
	elseif script.Parent.Value.Value == 4 then
		no4Txt = "2"
	end
	script.Parent.Value.Value = script.Parent.Value.Value + 1
	if script.Parent.Value.Value == 5 then
		script.Parent.Value.Value = 1
	end
end)

three.MouseButton1Click:Connect(function()
	if script.Parent.Value.Value == 1 then
		no1Txt = "3"
	elseif script.Parent.Value.Value == 2 then
		no2Txt = "3"
	elseif script.Parent.Value.Value == 3 then
		no3Txt = "3"
	elseif script.Parent.Value.Value == 4 then
		no4Txt = "3"
	end
	script.Parent.Value.Value = script.Parent.Value.Value + 1
	if script.Parent.Value.Value == 5 then
		script.Parent.Value.Value = 1
	end
end)

four.MouseButton1Click:Connect(function()
	if script.Parent.Value.Value == 1 then
		no1Txt = "4"
	elseif script.Parent.Value.Value == 2 then
		no2Txt = "4"
	elseif script.Parent.Value.Value == 3 then
		no3Txt = "4"
	elseif script.Parent.Value.Value == 4 then
		no4Txt = "4"
	end
	script.Parent.Value.Value = script.Parent.Value.Value + 1
	if script.Parent.Value.Value == 5 then
		script.Parent.Value.Value = 1
	end
end)

five.MouseButton1Click:Connect(function()
	if script.Parent.Value.Value == 1 then
		no1Txt = "5"
	elseif script.Parent.Value.Value == 2 then
		no2Txt = "5"
	elseif script.Parent.Value.Value == 3 then
		no3Txt = "5"
	elseif script.Parent.Value.Value == 4 then
		no4Txt = "5"
	end
	script.Parent.Value.Value = script.Parent.Value.Value + 1
	if script.Parent.Value.Value == 5 then
		script.Parent.Value.Value = 1
	end
end)

six.MouseButton1Click:Connect(function()
	if script.Parent.Value.Value == 1 then
		no1Txt = "6"
	elseif script.Parent.Value.Value == 2 then
		no2Txt = "6"
	elseif script.Parent.Value.Value == 3 then
		no3Txt = "6"
	elseif script.Parent.Value.Value == 4 then
		no4Txt = "6"
	end
	script.Parent.Value.Value = script.Parent.Value.Value + 1
	if script.Parent.Value.Value == 5 then
		script.Parent.Value.Value = 1
	end
end)

seven.MouseButton1Click:Connect(function()
	if script.Parent.Value.Value == 1 then
		no1Txt = "7"
	elseif script.Parent.Value.Value == 2 then
		no2Txt = "7"
	elseif script.Parent.Value.Value == 3 then
		no3Txt = "7"
	elseif script.Parent.Value.Value == 4 then
		no4Txt = "7"
	end
	script.Parent.Value.Value = script.Parent.Value.Value + 1
	if script.Parent.Value.Value == 5 then
		script.Parent.Value.Value = 1
	end
end)

eight.MouseButton1Click:Connect(function()
	if script.Parent.Value.Value == 1 then
		no1Txt = "8"
	elseif script.Parent.Value.Value == 2 then
		no2Txt = "8"
	elseif script.Parent.Value.Value == 3 then
		no3Txt = "8"
	elseif script.Parent.Value.Value == 4 then
		no4Txt = "8"
	end
	script.Parent.Value.Value = script.Parent.Value.Value + 1
	if script.Parent.Value.Value == 5 then
		script.Parent.Value.Value = 1
	end
end)

nine.MouseButton1Click:Connect(function()
	if script.Parent.Value.Value == 1 then
		no1Txt = "9"
	elseif script.Parent.Value.Value == 2 then
		no2Txt = "9"
	elseif script.Parent.Value.Value == 3 then
		no3Txt = "9"
	elseif script.Parent.Value.Value == 4 then
		no4Txt = "9"
	end
	script.Parent.Value.Value = script.Parent.Value.Value + 1
	if script.Parent.Value.Value == 5 then
		script.Parent.Value.Value = 1
	end
end)

x.MouseButton1Click:Connect(function()
	script.Parent.Parent.Visible = false
	no1Txt = "0"
	no2Txt = "0"
	no3Txt = "0"
	no4Txt = "0"
end)

finish.MouseButton1Click:Connect(function()
	if no1Crct == no1Txt then
		no1Correct = true
	end
	if no2Crct == no2Txt then
		no2Correct = true
	end
	if no3Crct == no3Txt then
		no3Correct = true
	end
	if no4Crct == no4Txt then
		no4Correct = true
	end
	
	if no1Correct and no2Correct and no3Correct and no4Correct == true then
		script.Parent.AllCorrect.Value = true
		no1Txt = "0"
		no2Txt = "0"
		no3Txt = "0"
		no4Txt = "0"
		
	else
		no1Txt = "0"
		no2Txt = "0"
		no3Txt = "0"
		no4Txt = "0"
		wait(.1)
		no1Txt = " "
		no2Txt = " "
		no3Txt = " "
		no4Txt = " "
		wait(.1)
		no1Txt = "0"
		no2Txt = "0"
		no3Txt = "0"
		no4Txt = "0"
		script.Parent.Value.Value = 1
	end
end)

thanks!