Color Changing meshparts

while making my suitcase gui i tested and color change is not working.
every time i click it just makes it black no matter what.

ServerScipt:

local configurations = require(script.Parent.Configure)

script.Parent.AncestryChanged:Connect(function(child,Newparent)
	if not Newparent.Parent.Name == "Workspace" then
		if game.Players:GetPlayerByUserId(Newparent.Parent.UserId) then
			local default1 = configurations.defaultColor1
			local default2 = configurations.defaultColor2
			local default3 = configurations.defaultColor3
			local default4 = configurations.defaultColor4
			local freeFullAccess = configurations.freeFullColorAccess
			local gamepassid = configurations.gamepassIdForFullColorAccess
			local plr = Newparent.Parent
			local mps = game:GetService("MarketplaceService")

			script.Parent.Frame.Default1.BackgroundColor3 = Color3.fromRGB(default1)
			script.Parent.Frame.Default2.BackgroundColor3 = Color3.fromRGB(default2)
			script.Parent.Frame.Default3.BackgroundColor3 = Color3.fromRGB(default3)
			script.Parent.Frame.Default4.BackgroundColor3 = Color3.fromRGB(default4)

			if freeFullAccess == true or gamepassid == 0 then
				script.Parent.Frame.FullColors.Lock.Visible = false
			else
				if mps:UserOwnsGamePassAsync(plr.UserId,gamepassid) then
					script.Parent.Frame.FullColors.Lock.Visible = false
				else
					script.Parent.Frame.FullColors.Lock.Visible = true
				end
			end
		else
			print("Not valid player.. Waiting to be parented to a player...")
		end
	end
end)

Configurations script:

local config = {}
config.freeFullColorAccess = true -- Will full color be free? If yes then set to true if no then set to false.
config.gamepassIdForFullColorAccess = 0 -- If freeFullColorAccess is false then insert a gamepass id here that people must buy to use full colors! (If left at 0 freeFullColorAccess will be turned on by default!)
config.defaultColor1 = "255, 0, 0"
config.defaultColor2= "0, 0, 255"
config.defaultColor3 = "0, 255, 0"
config.defaultColor4 = "255, 255, 0"
return config

You cant use strings for Color3, make them in the configuration instead of the script.

Server script:

-			script.Parent.Frame.Default1.BackgroundColor3 = Color3.fromRGB(default1)
-			script.Parent.Frame.Default2.BackgroundColor3 = Color3.fromRGB(default2)
-			script.Parent.Frame.Default3.BackgroundColor3 = Color3.fromRGB(default3)
-			script.Parent.Frame.Default4.BackgroundColor3 = Color3.fromRGB(default4)

+			script.Parent.Frame.Default1.BackgroundColor3 = Color3.fromRGB(configurations.defaultColor1)
+			script.Parent.Frame.Default2.BackgroundColor3 = Color3.fromRGB(configurations.defaultColor2)
+			script.Parent.Frame.Default3.BackgroundColor3 = Color3.fromRGB(configurations.defaultColor3)
+			script.Parent.Frame.Default4.BackgroundColor3 = Color3.fromRGB(configurations.defaultColor4)

Configuration:

local config = {}
config.freeFullColorAccess = true -- Will full color be free? If yes then set to true if no then set to false.
config.gamepassIdForFullColorAccess = 0 -- If freeFullColorAccess is false then insert a gamepass id here that people must buy to use full colors! (If left at 0 freeFullColorAccess will be turned on by default!)
config.defaultColor1 = Color3.fromRGB(255, 0, 0)
config.defaultColor2= Color3.fromRGB(0, 0, 255)
config.defaultColor3 = Color3.fromRGB(0, 255, 0)
config.defaultColor4 = Color3.fromRGB(255, 255, 0)
return config

Nope, it didnt work.