opensalt icon indicating copy to clipboard operation
opensalt copied to clipboard

EducationLevel not imported from CSV

Open brandonopened opened this issue 6 years ago • 7 comments

Describe the problem

Education Level is not getting loaded on a CSV import.

Version of OpenSALT

2.1

To reproduce

Steps to reproduce the behaviour:

  1. Go to a new framework
  2. Click on Import Children
  3. If the EducationLevel field has a grade level on it such as 01, 02 etc.
  4. The Grade level is not imported.

Expected behaviour

A clear and concise description of what you expected to happen.

Actual behaviour

A clear and concise description of what actually occurs.

Screenshots

If applicable, add screenshots to help explain your problem.

Your system (please complete the following information):

  • Device: [e.g. desktop, iPhone6]
  • OS: [e.g. iOS8.1, WIndows 10]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

If you are able to report the URL where an error happened or output from the dev console please do so.

Additional acceptance criteria tests

Scenario:  
Given 
When 
Then 

Additional context

Add any other context about the problem here.

brandonopened avatar Aug 20 '18 19:08 brandonopened

We've gone over this before; sometimes in the code I see EducationLevel, other times EductionalAlignment. I think the CSV importer is using the wrong one so should be a simple fix!

brandonopened avatar Aug 21 '18 15:08 brandonopened

In the CASE export EducationLevel looks like this: "educationLevel":["05"]

I tried putting this in the import file (both "05" and ["05"] in the EducationLevel field to no avail...

brandonopened avatar Aug 21 '18 15:08 brandonopened

EducationLevel works in Spreadsheet updater. Until we incorporate SequenceNumber into Spreadsheet updater though it can't act as the primary import vehicle.

brandonopened avatar Aug 21 '18 18:08 brandonopened

It appears that the (now poorly named) GithubImport::parseCSVGithubStandard() method does not even look at the educationLevel field.

roverwolf avatar Aug 21 '18 19:08 roverwolf

FYI: I'm not sure this a BC Break, Georgia DOE has now reported this issue in v1.3.

roverwolf avatar Aug 23 '18 14:08 roverwolf

If this is just the CSV we might move it out of scope for 2.3 unless Bandon also wants to perfect the spreadsheet (Not CSV) import.

jmarks avatar Sep 30 '19 21:09 jmarks

CSVs are no longer support change this to Spreadsheet.

jmarks avatar Sep 30 '19 21:09 jmarks