Closed erickpeirson closed 6 years ago
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
submit/controllers/util.py | 4 | 7 | 57.14% | ||
submit/controllers/classification.py | 53 | 71 | 74.65% | ||
submit/routes/ui.py | 0 | 22 | 0.0% | ||
<!-- | Total: | 61 | 104 | 58.65% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
submit/routes/ui.py | 1 | 0.0% | ||
<!-- | Total: | 1 | --> |
Totals | |
---|---|
Change from base Build 106: | -1.1% |
Covered Lines: | 333 |
Relevant Lines: | 514 |
Going to merge, but welcome any additional comments post-merge
This adds cross-list functionality to the submission UI. Things that this PR does now:
Things that this feature does not do:
Note: In this implementation, if the user has selected a category but has not clicked the "add" button, I am not saving the result on previous/next. This seemed the most intuitive to me, but please feel free to resist that. This is a complex feature in some ways; let's consider all options.
@eawoods I'm looking to you primarily for feedback on expected UX/functionality and layout/appearance.
@DavidLFielding Adding you for an extra set of eyes, if you get a chance.
I'd like to merge this by noon tomorrow (Thursday, 21 June), so that we can keep rolling.