Improved the form's clean method so that errors would be associated with their fields.
Added a check to the form's clean method to ensure that the files are excel and that the peak annotation files were specifically either accucor or isocorr (individually).
Changed the order of the set_files args and made the sample arg optional.
Added methods: is_accucor and is_isocorr to the AccucorDataLoader class.
Changed the way things were imported from django.forms so that it would look cleaner.
Affected Issues/Pull Requests
Partially addresses #829
Merges into PR #924
Next PR: #926
Review Notes
See comments in-line.
Checklist
This pull request will be merged once the following requirements are met. The
author and/or reviewers should uncheck any unmet requirements:
Review requirements
Minimum approvals: 1
No changes requested
All blocking issues resolved by reviewers
Specific reviewers: @add_username_here
Review period: 2 days
Associated issue/pull request requirements:
[x] All requirements in affected issues marked "resolved" are satisfied
[x] All required pull requests are merged (or none)
Summary Change Description
Affected Issues/Pull Requests
Review Notes
See comments in-line.
Checklist
This pull request will be merged once the following requirements are met. The author and/or reviewers should uncheck any unmet requirements:
changelog.md
(or no change)