Confirm source for generating course title and tag in enrolment status modal #5

New issue

Closed

opened 2018-11-25 12:36:01 +13:00 by mackiwg · 4 comments

mackiwg commented

2018-11-25 12:36:01 +13:00

(Migrated from git.oeru.org)

It appears that the enrollment status modal is generating the "course title" string from the WordPress Site Title field to produce for example, the following text in the modal.

"You are enrolled in "Learning in A Digital Age 102 (dev)" followed by the course code, eg (lida102)

This works fine on wpms, but on course.oeru.org the Site Title field is usually the course code so depending on where you are sourcing the course code string from the database there will be duplication in the model text, for example on course.oeru.org the text in the modal would read:

"You are enrolled in LiDA101 (lida102)"

Suggested quick fix: Remove the second course code in parenthesis i.e. "(lida102)" - check that this change won't mess up the button text: Enrol in lida102 (Not sure where you are getting the course code from in the DB.

This issue would apply to all scenarios where the modal pull the Site title field, eg:

Enrol in this OERu Course modal
Notifications which use the Site title field
Others?

It appears that the enrollment status modal is generating the "`course title`" string from the WordPress `Site Title` field to produce for example, the following text in the modal. "You are enrolled in "Learning in A Digital Age 102 (dev)" followed by the course code, eg (lida102) This works fine on wpms, but on course.oeru.org the `Site Title` field is usually the course code so depending on where you are sourcing the course code string from the database there will be duplication in the model text, for example on course.oeru.org the text in the modal would read: "You are enrolled in LiDA101 (lida102)" Suggested quick fix: Remove the second course code in parenthesis i.e. "(lida102)" - check that this change won't mess up the button text: `Enrol in lida102` (Not sure where you are getting the course code from in the DB. This issue would apply to all scenarios where the modal pull the `Site title` field, eg: * Enrol in this OERu Course modal * Notifications which use the `Site title` field * Others?

mackiwg commented

2018-11-25 12:50:06 +13:00

(Migrated from git.oeru.org)

changed title from Confirm source for generating course title and tag in enrol{-l-}ment status modal to Confirm source for generating course title and tag in enrolment status modal

changed title from **Confirm source for generating course title and tag in enrol{-l-}ment status modal** to **Confirm source for generating course title and tag in enrolment status modal**

mackiwg commented

2018-11-25 13:19:49 +13:00

(Migrated from git.oeru.org)

changed the description

dave commented

2018-11-26 15:54:35 +13:00

(Migrated from git.oeru.org)

I've removed the course code listing (in the ()) as you suggest. It was mostly there to ensure the user was clear for which course they were enrolled (or not)...

dave commented

2018-11-26 15:58:31 +13:00

(Migrated from git.oeru.org)

closed