local PrimaryColor = Instance.new("BrickColorValue",Storage)
PrimaryColor.Name = "PrimaryColor"
PrimaryColor.Value = PrimaryColor1:GetAsync(player.UserId) or BrickColor.Random()
PrimaryColor.Changed:connect(function()
PrimaryColor1:SetAsync(player.UserId,PrimaryColor.Value)
end)
At line 3.

5 comments
Thanks beforehand xd
It would be nice to see the color value modified before the script. I can't really understand the issue fully yet.
You can't store userdatas in data stores. A BrickColor is considered a userdata. You may want to save the name of the BrickColor (PrimaryColor.Value.Name), BrickColors have a Name property.
The constructor for the brick color would then be BrickColor.new(PrimaryColor.Value.Name).
Oh, I’ll try this method when I get home. Thx