Can someone please tweak my script and tell me what I did wrong? (I'm trying to make a shop GUI)

I see what you are trying to do, and see no point in not helping out.

local Frame = script.Parent.Parent.Frame
local OpenButton = script.Parent
local Open = false

OpenButton.MouseButton1Click:Connect(function()
    if Open == false then
        Open = true
        --[[Optional:
        Open.Text = "Close Shop"
        ]]--
        Frame.Visible = true
    elseif Open == true then
        Open = false
        --[[Optional:
        Open.Text = "Open Shop"
        ]]--
        Frame.Visible = false
    end
end)

To further explain the script, it is a open and close script that should be placed in the button, and also make sure that the frame and button are in the correct places.

Hope this helps, Happy scripting!

please say answered