That looks OK.
The only thing is that you have a deDE for a only one entry in addon.db where
Code:
tinsert(data, {item.announce[GetLocale()], item.icon, item.name})
will cause an error if the entry is missing in any single entry for the users locale return from GetLocale().
Have Fun!