callisto-core icon indicating copy to clipboard operation
callisto-core copied to clipboard

rename "delivery" app to something more relevant

Open kelseyq opened this issue 9 years ago • 4 comments

It does deliver reports, but it also stores them encrypted and matches them.

kelseyq avatar May 26 '16 06:05 kelseyq

processing or report_processing? Or does that encroach on what the evaluation app does?

kevindaum avatar May 26 '16 14:05 kevindaum

I think "reports" might be best, since it processes, stores, and delivers them. It's possible it should be more than one app, although I don't think any one part of that would be useful without the others--still learning the dark arts of Django app delineation.

Speaking of, one wrinkle with this bug (for me) is that I use the app in our production instances of Callisto, so we have tables named "delivery_" in prod. Any thoughts on a reasonable way to rename the app while keeping the data in those tables around? This seems kind of scary, and I think dependencies would scotch it anyway. I'm not above creating a legacy branch if need be.

kelseyq avatar May 26 '16 22:05 kelseyq

You know, I can't think of an app I've renamed after putting a project into production. I have one I might rename this summer, though, so I may get some experience with it. That said, the combination of careful migrations and database backups should make the process possible without fear of data loss. I'll take a look at Markus' recipe (maybe next week since it's a holiday weekend, family, etc.), and try to figure out how scary it really is.

kevindaum avatar May 27 '16 21:05 kevindaum

~~I'm releasing 1.0 in this PR https://github.com/SexualHealthInnovations/callisto-core/pull/202 so app renames are out of scope~~

coilysiren avatar Jun 24 '17 02:06 coilysiren