Merge pull request #2256 from edx/diana/linkedin-parameter
Remove authority name from url.
This commit is contained in:
@@ -167,7 +167,6 @@ class Command(BaseCommand):
|
||||
'gf'])
|
||||
query = [
|
||||
('pfCertificationName', course.display_name_with_default),
|
||||
('pfAuthorityName', settings.PLATFORM_NAME),
|
||||
('pfAuthorityId', settings.LINKEDIN_API['COMPANY_ID']),
|
||||
('pfCertificationUrl', certificate.download_url),
|
||||
('pfLicenseNo', certificate.course_id),
|
||||
|
||||
@@ -165,7 +165,7 @@ class MailusersTests(TestCase):
|
||||
self.assertEqual(
|
||||
fut(self.cert1),
|
||||
'http://www.linkedin.com/profile/guided?'
|
||||
'pfCertificationName=TEST1&pfAuthorityName=edX&'
|
||||
'pfCertificationName=TEST1&'
|
||||
'pfAuthorityId=0000000&'
|
||||
'pfCertificationUrl=http%3A%2F%2Ftest.foo%2Ftest&pfLicenseNo=TESTX%2F1%2FTEST1&'
|
||||
'pfCertStartDate=201005&_mSplash=1&'
|
||||
|
||||
Reference in New Issue
Block a user