Discord -> Webhook Embed Attempt to concatenate nil with string

Help and Feedback Scripting Support
#1

Hiya devs, when I try send an embeded message to Discord, I get this error;

image
image877×67 10.9 KB

But the thing is that in the local script, the string will always equal a number between 1 and 5 not matter what so I have no idea why it wont work.

Server Script;

local DiscordWebhook = "" -- Removed so I dont get raided (Will replace when testing)
local HTTPSservice = game:GetService("HttpService")
local Event = game.ReplicatedStorage.RatingSystem.Ratings

Event.OnServerEvent:Connect(function(Player, AirportRating, ServiceRating, AircraftRating, Suggestion)
	local Data = 
		{
			["content"] = "",
			["embeds"] = {{
				["title"] = Player.Name,
				["description"] = "Ratings:		Airport: "..AirportRating.."/5",


			}}
		}
	Data = HTTPSservice:JSONEncode(Data)
	HTTPSservice:PostAsync(DiscordWebhook, Data)
end)

Local Script;

local Event = game.ReplicatedStorage.RatingSystem.Ratings

local AirportRatingUi = script.Parent.AirportRating
local ServiceRatingUi = script.Parent.ServiceRating
local AircraftRatingUi = script.Parent.AircraftRating

local AirportRating = AirportRatingUi.Rating.Value
local ServiceRating = ServiceRatingUi.Rating.Value
local AircraftRating = AircraftRatingUi.Rating.Value
local Suggestion = script.Parent.Suggestion.Text
local submit = script.Parent.Submit

submit.MouseButton1Click:Connect(function(Player)
	Event:FireServer(Player, AirportRating, ServiceRating, AircraftRating, Suggestion)
end)

Line 12 is this line;

         ["description"] = "Ratings:		Airport: "..AirportRating.."/5",
#2

you get the value of airportrating when the function first runs, it will never change and will always be empty. you should get the value on mouse button click

#3

You’re referencing the values from at the start of the UI, not when they actually press submit. Change your script to this:

local Event = game.ReplicatedStorage.RatingSystem.Ratings

local AirportRatingUi = script.Parent.AirportRating
local ServiceRatingUi = script.Parent.ServiceRating
local AircraftRatingUi = script.Parent.AircraftRating

local AirportRating = AirportRatingUi.Rating
local ServiceRating = ServiceRatingUi.Rating
local AircraftRating = AircraftRatingUi.Rating
local Suggestion = script.Parent.Suggestion
local submit = script.Parent.Submit

submit.MouseButton1Click:Connect(function(Player)
	Event:FireServer(Player, AirportRating.Value, ServiceRating.Value, AircraftRating.Value, Suggestion.Text)
end)
#4

I still get the same error,
I tried formatting it differently, and still didnt work I get the same error;

submit.MouseButton1Click:Connect(function(Player)
	local AirportRatingMain = AirportRating.Value
	local ServiceRatingMain = ServiceRating.Value
	local AircraftRatingMain = AircraftRating.Value
	local SuggestionMain = AircraftRating.Value
	
	Event:FireServer(Player, AirportRatingMain, ServiceRatingMain, AircraftRatingMain, SuggestionMain)
end)
#5

Oh, I just realized that you’re trying to get the player from a MouseButton1Click and sending it over to the server. You don’t have to send the player instance over, as the Server already knows who sent the event.

Change your local script to this:

local Event = game.ReplicatedStorage.RatingSystem.Ratings

local AirportRatingUi = script.Parent.AirportRating
local ServiceRatingUi = script.Parent.ServiceRating
local AircraftRatingUi = script.Parent.AircraftRating

local AirportRating = AirportRatingUi.Rating
local ServiceRating = ServiceRatingUi.Rating
local AircraftRating = AircraftRatingUi.Rating
local Suggestion = script.Parent.Suggestion
local submit = script.Parent.Submit

submit.MouseButton1Click:Connect(function()
	Event:FireServer(AirportRating.Value, ServiceRating.Value, AircraftRating.Value, Suggestion.Text)
end)