Provide CTA to View Certificate on Learning Path home

Hi @butlerbryanc

In the attached patch, I see

+    $pdf_route = 'certificate.entity.pdf';

However, I can't find this route in opigno_learning_path.routing.yml. Is this route defined in another opigno routing file?

Regarding this change:

+    if (!empty($is_passed)) {
+      $buttons[] = $view_certificate_button;
+    }

It appears that $is_passed is set to a boolean value.

$is_passed = opigno_learning_path_is_passed($group, $uid);

The function docblock, defined in opigno_learning_path.module, states:

/**
 * Returns LP passed flag.
 *
 * @param array|\Drupal\group\Entity\GroupInterface $step
 *   Learning Path or Course Group entity,
 *   or step array, returned by opigno_learning_path_get_steps()
 * @param int $uid
 *   User ID.
 *
 * @return bool
 *   LP passed flag.
 *
 * @throws \Drupal\Component\Plugin\Exception\InvalidPluginDefinitionException
 * @throws \Drupal\Component\Plugin\Exception\PluginException
 * @throws \Drupal\Component\Plugin\Exception\PluginNotFoundException
 */
function opigno_learning_path_is_passed($step, $uid) {

Rather than checking,

if (!empty($is_passed)) {}

I think it would be more straight forward if we used,

if ($is_passed) {}

does that make sense?

Additionally, it appears that this feature doesn't include functionality for an administrator to toggle, is that right?

I've unassigned this issue.

This patch does not currently apply to 8.x-1.12 version.

I think this patch altered the LearningPathController::progress() method. However, this was removed in this commit > https://git.drupalcode.org/project/opigno_learning_path/-/commit/acf5a3e...

Is there an alternative to LearningPathController::progress()

Hi,
We are working on a complete redesign of UI with Opigno 3.0 (Drupal 9), in order to integrate a lot of feedbacks from users and improve UX.
Among these feedbacks we considered this one, and we will add several additional CTA allowing to get more direct access to certificates, without visiting achievements page.
This will be introduced at the latest at the beginning of September, so coming soon.

Thanks for the info.

Regarding my question,

Is there an alternative to LearningPathController::progress()

Could you point me to a replacement? Is there something that can be used to report a student's progress on the course detail page?