-
Notifications
You must be signed in to change notification settings - Fork 10
Fix bug 1065624: Remove subscriber data from db. #142
Conversation
|
||
operations = [ | ||
migrations.DeleteModel( | ||
name='Subscriber', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I recommend moving this to a separate commit, to be deployed after the removal from news/models.py
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done.
c2731ea
to
7caf658
Compare
Subscriber data should only be in ExactTarget.
Remove nose et. al.
0fe3129
to
2184809
Compare
@jgmize I think I've addressed your concerns so far. Thanks for the review. |
@pmclanahan would you mind excluding the migrations folder from the coverage check? |
Ignore migrations in coverage reports.
2184809
to
b9853bb
Compare
I've updated the coverage config. Seems to be working. @jgmize you think this is good enough now to merge into |
Yes, this lgtm for merging into the 12factor branch. Nice work :) |
Fix bug 1065624: Remove subscriber data from db.
Replaces #141 and is based on WIP branch (12factor) for work
related to readying the site for deployment to heroku or deis.