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)