- Issue created by @socketwench
- 🇺🇸United States socketwench
This patch adds a check to complete the Learning Path in the form. This works, but as I borrowed this from OpignoModuleController::userStatus(), I feel the problem is architectural. Either way, this works to correct the fault for manual scoring.
- 🇺🇸United States socketwench
This updated patch corrects a fault where the step achievements also do not score correctly, resulting in an issue where the opigno_statistics.training route does not display the expected completion amount.
- 🇺🇸United States socketwench
In https://www.drupal.org/project/opigno_learning_path/issues/3523674 🐛 LpSteps::getCourseStep should include max score Active it was discovered that the completion logic still doesn't work as expected. This patch corrects the issue (hopefully for the last time) along with the referenced opigno_learning_path patch.
- 🇺🇸United States socketwench
In https://www.drupal.org/project/opigno_learning_path/issues/3524000 🐛 LpSteps::getCourseStep should include attempts allowed Active we also found out that we need to compare the number of attempts compared to the maximum allowed attempts for each mandatory step to be sure the LP should be marked as "failed" rather than just "incomplete".
This patch updates the logic to include the fix, but also requires the patch from the above issue in order to work.