Thomas Debay

Results 54 comments of Thomas Debay

Hello @Dummdevka ! Yes, you're very welcome to work on this feature. You can see how charts are built in the [latest views](https://github.com/academico-sis/academico/blob/master/resources/views/reports/gender.blade.php) and reproduce something similar (these are built...

- [ ] Improve result.index - [x] Improve result.show (see comments in the code) - [ ] list all enrolments with no result on teacher dashboard. - [x] Allow course...

- [ ] Compute a percentage according to the skills evaluation for each student

Hello @iphadey and thank you for this great PR. It works great. I only have one concern before merging it. The outputs for these two -hypothetical- situations would be identical:...

Hello @henok-tesfu and thank you for your interest in the project. Yes, you're very welcome to work on this feature. I'll assign it to you, please let me know if...

Also, "show" views should benefit from backpack's features

Hi @Zen-glen and thank you for reporting this issue. I'll see what I can do about it. I'm curious to know if you're using this software and in which context?...

I'll take more time to review your code more in depth. From what I've seen, it works great and you've covered all the use cases. I think we could tweak...

Also, it's important to check if `$courseTimes` is null otherwise it will throw an error when at `foreach()`

> How should it work in this case? https://github.com/academico-sis/academico/blob/oss/tests/Unit/CourseTest.php#L187-L208 I would say that it should stay exactly as you suggested. Unless all the course times are exactly the same throughout...