Skip to content

Fix course url link in enrollment emails. - #357

Merged
dcadams merged 2 commits into
masterfrom
bugfix/dcadams/enroll_emails
Jul 10, 2013
Merged

Fix course url link in enrollment emails.#357
dcadams merged 2 commits into
masterfrom
bugfix/dcadams/enroll_emails

Conversation

@dcadams

@dcadams dcadams commented Jul 9, 2013

Copy link
Copy Markdown
Contributor

Course link within the enrollment emails was being constructed incorrectly.

@ghost ghost assigned dianakhuang Jul 9, 2013
@dianakhuang

Copy link
Copy Markdown
Contributor

Any way you could write a test to verify that the urls will continue to be generated properly in the future?

@dianakhuang

Copy link
Copy Markdown
Contributor

Looks good. 👍

dcadams pushed a commit that referenced this pull request Jul 10, 2013
Fix course url link in enrollment emails.
@dcadams
dcadams merged commit a5f8607 into master Jul 10, 2013
@dcadams
dcadams deleted the bugfix/dcadams/enroll_emails branch July 10, 2013 16:47
chrisrossi pushed a commit to jazkarta/edx-platform that referenced this pull request Mar 31, 2014
fix typo in registration button jquery
antoviaque referenced this pull request in open-craft/openedx-platform Dec 29, 2014
Yasui-ks added a commit to nttks/edx-platform that referenced this pull request Oct 6, 2015
Sujeet1379 pushed a commit to chandrudev/edx-platform that referenced this pull request Nov 17, 2022
Danyal-Faheem pushed a commit to Danyal-Faheem/edx-platform that referenced this pull request Jul 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants