search

AppStateESS / InternshipInventory

Web app for tracking and approving student internships.
http://appstateess.github.io/InternshipInventory/
GNU General Public License v3.0
9 stars 22 forks source link

Restore major fields in CSV export #370

Closed jlbooker closed 6 years ago

jlbooker commented 7 years ago

In Internship::getCsv(), we need to use the major_description field to output the undergrad or graduate major (as determined by the level field). Be sure this is consistent when the major/level is not selected, and that the proper number of fields are always in the CSV output (so columns do not become misaligned).

caldwellc1 commented 7 years ago

@jlbooker I think it might be better to fix this problem in the issue where I created the generic student types. It still has the same problem though since I based it on the previous code and did not fully check the csv as I now know how to. Since it hasn't been merged yet I can still add it to the pull request for that issue.