Roster import multiusr - Githubissues

DASSL / Gradebook

Open-source product to provide a practical means for instructors to record student attendance and assessment

Other

8 stars 4 forks source link

Closed afig closed 7 years ago

afig commented 7 years ago

Remove single-user version of roster import and modify multi-user version to use the Gradebook schema and a temporary staging table.

Improvements were also made to the readability (comments) of the batch file.

smurthys commented 7 years ago

importRosterCSV.bat:

The order of some of the comment lines seem different from the order used elsewhere
On Line 24, should that be %5 instead of %4?
Need clear comments on --singletransaction switch: it is key to multi-user operation

rosterImport.sql:

smurthys commented 7 years ago

I wonder if --singletransaction also includes the effect of ON_ERROR_STOP. If it doesn't, we will need to add it.