Having trouble with RemoteFunctions

[OverAcheiver647 (BothAchiever647_YT)]

I’m wanting to have a remotefunction that returns a boolean.
this error is in the output:

and here is the localScript that may be causing it

local event = game.ReplicatedStorage.RequestPrompt
local Donator = game.ReplicatedStorage.Donators.RequestBase
local Frame = script.Parent
local plr = game.Players.LocalPlayer

event.OnClientInvoke:Connect(function(plr2)
	local Return = nil
	local clone = Donator:Clone()
	clone.Parent = Frame
	clone.PlayerLabel.Text = plr2 .." Party Up?"
	local Accept = clone.Accept
	local Decline = clone.Decline
	Accept.Activated:Connect(function()
		Return = true
		clone:Destroy()
	end)
	Decline.Activated:Connect(function()
		Return = false
		clone:Destroy()
	end)
	clone.Destroying:Connect(function()
		return Return
	end)
end)

[ITemply (tem)]

try:

event.OnClientInvoke = function(plr2)

instead of:

event.OnClientInvoke:Connect(function(plr2)

[OverAcheiver647 (BothAchiever647_YT)]

It fixed the error, but it always returns as nothing. anyway to fix that?

[ITemply (tem)]

@OverAcheiver647

Accept.Activated:Connect(function()
		Return = true
		clone:Destroy()
	end)

do the return like here and get rid of the destroying part

Accept.Activated:Connect(function()
	Return = true
        return Return
	clone:Destroy()
end)

(maybe, u might wanna check if everything is activating too)

[OverAcheiver647 (BothAchiever647_YT)]

it seems to have this error:

[gamincwc (Dejan)]

show a ss of the latest code so I can see where you are missing a end

[iondrive]

You have a line of code after the return:

return Return
	clone:Destroy()

The clone:Destroy() call will never run as the function will return before it.

[OverAcheiver647 (BothAchiever647_YT)]

how would i get the instance to destroy then?

[gamincwc (Dejan)]

I don’t know what you are talking about rn because I don’t have an idea of what the script looks like

[OverAcheiver647 (BothAchiever647_YT)]

[gamincwc (Dejan)]

try instead of function accept() do local function accept() and same with the other one

[OverAcheiver647 (BothAchiever647_YT)]

that changed nothing @gamincwc

[gamincwc (Dejan)]

what is Accept, a gui button? oh and also you don’t need the Return variable

[OverAcheiver647 (BothAchiever647_YT)]

what? is there another way to get a bool from the remotefunction?

[gamincwc (Dejan)]

the Accept I assume is a button so try instead of .Activated do .MouseButton1Clicked
.Activated is not even used that much as it is a property and not an event

[OverAcheiver647 (BothAchiever647_YT)]

I fixed it by putting clone:destroy() before return and i added

repeat wait() until Return ~= nil
	return Return

to the end of the OnClientInvoke event.