You should be assigning the parent property last, not directly after the instance is cloned.
local newlog = template:Clone()
newlog.Name = "Log"..count
newlog.Text = name..": "..typer.Value.." | "..victim.Value
newlog.Visible = true
newlog.Parent = script.Parent -- Or wherever else the frame that holds it is.