operepo / ope

Open Prison Education project - Code and scripts to enable offline docker services and offline laptop syncing for inmate education
MIT License
14 stars 6 forks source link

*Temp. work around* Assignment lists in LMS app not in order #106

Open mdhuse opened 2 years ago

mdhuse commented 2 years ago

Have been contacted several times about list of assignments in the LMS app not showing in order. For example a list would show:

Assignment 04 Assignment 05 Assignment 06 Assignment 01 Assignment 02 Assignment 03

I will try to get a screen shot for better visual.

jmichel-sbctc commented 2 years ago

If you can get a screenshot that would be great. I'd be interested to know how the assignments are setup in canvas as well. If you were able to get me a copy of the course I could look at it more closely.

I'm wondering if something like a leading space on the assignment name has caused the issue, or capital verses non-capital letter. I'm going to try and test out if these cause it from my end when I have a chance.

jmichel-sbctc commented 2 years ago

Checked various settings and everything seemed to be correct. Also check assignment ID and nothing seemed to indicate that the strange ordering on the OPE-LMS app was related.

Putting the assignments into a modules list allows for keeping the correct flow of assignments, you can also change the order by rearranging the modules and their assignments\files (Syncing needs to happen to pull changes to laptops when they occur).

Still working to track down what is causing the assignments page not display assignments in the order assigned in canvas.

jmichel-sbctc commented 2 years ago

The issue is caused by how sql is handling the postion field of the database. It is sorting it as a string instead of a number.

1, 10, 11, 12, 2, 22, 24 etc.

Will need to apply the patch to all tables and will have a fix pushed out in a future release.