search

SuffolkLITLab / docassemble-InterviewStats

A docassemble extension.
MIT License
1 stars 0 forks source link

add-data-report-feature #15

Closed purplesky2016 closed 2 years ago

purplesky2016 commented 2 years ago

Fix issue #14

purplesky2016 commented 2 years ago

Latest changes:

  1. stats.yml

    • added an intro page to direct users to either Stats or Data Reporting;
    • added an error check to prevent selecting any interview with the title column, which is a sign that the user should run Data Reporting

  2. data_report.yml - added error checks for no-data and no-reportable data

  3. The 2 test_xxx.yml files - added a link to its applicable feature

  4. data_report.py - excluded numeric fields

I am done with my changes. This PR is ready for review now.

nonprofittechy commented 2 years ago

Thanks @purplesky2016 we talked this over a bit on Monday and I think the only substantive change is that we want to remove the kickout screen if there is a title column in the interview. I can plan to look at this and merge it soon after removing that kickout -- it's not necessary for @purplesky2016 to do it.