Conversation
The student statistics and student progress page are identical. Remove the student statistics version of the page, since the page doesn't provide any summary of the data, and only shows a student's number of attempts and grade.
|
I haven't looked over the code thoroughly yet, but I can already see how much this cleans up the mess of trying to share the templates between the stats and progress pages that I had to go through when I converted to using Mojolicious templates. I will try to look the code over better later, but I am sure it is generally good. |
6f609fb to
a678fbf
Compare
|
@drgrice1 I noticed that the siblings template was also shared. This one was a little harder for me to decide if it should be split into two places or not. There are differences between then (stats will never need to create a student list, while student progress never creates problem list), but there is also a lot of overlap too. I split them just to try to limit sharing (there really isn't a centralized place to share templates), it also keeps from having to find/replace the |
a678fbf to
fe4393c
Compare
|
I went with only sharing the code that links to the sets, but the unshared code is in individual templates. |
fe4393c to
a533302
Compare
The only shared code is a new template which just links to all sets. The problem links or student links are put in each pages siblings template.
a533302 to
a8d3542
Compare
The student statistics and student progress page are identical. Remove the student statistics version of the page, since the page doesn't provide any summary of the data, and only shows a student's number of attempts and grade.