Remove emergency alerts code#3976
Conversation
|
At the moment if you try to view a broadcast template in the admin app you will get an exception (the app doesn’t know how to preview one of these templates). But the only way you can get to that page is by manually entering the We could put in a workaround to return a |
|
i think to start with we could put 404s on any broadcast template or broadcast service endpoint, and then delete the data later i'd be keen on deleting the data in the DB earlier rather than later so we dont get surprised by weird data if we want to update schemas on those tables or migrate data based on columns listed in the models or things like that |
|
FYI, we can't delete any of the broadcast data out of our database for 1 year please. Either that or we need to take a snapshot of that data and store it elsewhere for a year. This is because emergency alerts need it as an audit trail for any security incidents (precedent is to store it for a year, just like Notify does). The 1 year is from the time they migrated to their own system (was that like early November maybe?) |
|
thanks, that's useful to know david. I'd probably be keen on snapshotting and storing it in a pgdump in s3 ideally |
bbf8b56 to
4bbec83
Compare
4bbec83 to
c6d66eb
Compare
c6d66eb to
85bbc98
Compare
9ce67d6 to
dd6210a
Compare
no longer used. there might be additional cleanup we can do of other utils functions that dont have "alert" or "broadcast" in the name that are no longer used
the schema was originally created so that it could not show content. However, then it needed content conditionally for broadcast messages so content was added back in. Now that broadcasts no longer exist we can revert that - however, for a template that has a specific list of expected keys, it's nicer to just list those rather than maintain a huge list of exceptions that needs to be modified every time a field changes on the template model
dd6210a to
e45afcd
Compare
e45afcd to
394d124
Compare
|
This has been checked and rebased to get rid of merge conflicts. The data has been deleted, so the comments relating to that no longer apply. The failure of the |
This doesn't deal with any database migration, but it does get rid of all code.
TODO before we merge: