What's wrong with this script?

local menu = game.Players.LocalPlayer.PlayerGui.Gui.Menu
local userInputService = game:GetService("UserInputService")
local menuOut = false
local cooldown = false

userInputService.InputBegan:Connect(function(input, isTyping)
    if isTyping then return
        else if cooldown == false and menuOut == false and input.KeyCode == Enum.KeyCode.M then
            game.ReplicatedStorage.RemoteEvent.MenuOut:FireClient()
            menuOut = true
            cooldown = true
            wait(1)
            cooldown = false
        else if cooldown == false and menuOut == true and input.KeyCode == Enum.KeyCode.M then
                game.ReplicatedStorage.RemoteEvent.MenuIn:FireClient()
                menuOut = false
                cooldown = true
                wait(1)
                cooldown = false
            end
        end     
    end
end)

The above script was meant to fire a remote event to a local script

local menu = script.Parent

game.ReplicatedStorage.RemoteEvent.MenuIn.OnClientEvent:Connect(function()
    menu:TweenPosition(UDim2.new(1,0, -0.05,0),"Out","Quad",0.5,false)
end)

game.ReplicatedStorage.RemoteEvent.MenuOut.OnClientEvent:Connect(function()
    menu:TweenPosition(UDim2.new(0.8,0, -0.05,0),"Out","Quad",0.5,false)
end)

When M was pressed, then the "menu" would move in/out

But it doesn't seems to work, can anyone help?