This pull request fixes the TODO mentioned in #149 -- it updates our other file-based import for email subscriptions to populate the new import_count field, added to the import_files table in #149.
It also adds the import options to the show view of an ImportFile, using a new partial to DRY.
Also modifies the user view to remove help text, which can be outdated if we purge the logs database.
How should this be reviewed?
👀
Any background context you want to provide?
Wanted to get this in before deploying #148 and #149 to avoid having any Email Subscription import files with a 0 value for their import_count, which would be confusing.
What's this PR do?
This pull request fixes the TODO mentioned in #149 -- it updates our other file-based import for email subscriptions to populate the new
import_countfield, added to theimport_filestable in #149.It also adds the import options to the
showview of an ImportFile, using a new partial to DRY.Also modifies the user view to remove help text, which can be outdated if we purge the logs database.
How should this be reviewed?
👀
Any background context you want to provide?
Wanted to get this in before deploying #148 and #149 to avoid having any Email Subscription import files with a 0 value for their
import_count, which would be confusing.Relevant tickets
Side effect of Pivotal #171848124.
Checklist