I am wondering if this is the right code for a ingame notifier?

I am making a ingame item notifier and I am wondering if this is the right http code?

Http = game:GetService"HttpService"
while wait(1) do
	DataTaken = Http:GetAsync("http://roproxy.tk/catalog/json?Subcategory=17&SortType=3&IncludeNotForSale=true&ResultsPerPage=3")
	Data = Http:JSONDecode(DataTaken)
	for Num in pairs(Data) do
		if script.Parent.Frame[Num].AssetId.Value ~= Data[Num]["AssetId"] then
		script.Parent.Frame[Num].AssetId.Value = Data[Num]["AssetId"]
		if script.Parent.Frame.Visible == false then
		script.Parent.OpenNClose.TextButton.Text = "!"
		script.Parent.OpenNClose.BackgroundColor3 = Color3.new(255,0,0)
		end
		end
	end
end

Please answer (I KNOW THIS ISNT THE ONLY SCRIPT JUST THE MAIN)