Skip to content

PdfViewer: Added missing link to compatibility.js - #1122

Merged
symbolist merged 1 commit into
masterfrom
usman/lms1167-pdfjs-update
Sep 27, 2013
Merged

PdfViewer: Added missing link to compatibility.js#1122
symbolist merged 1 commit into
masterfrom
usman/lms1167-pdfjs-update

Conversation

@symbolist

Copy link
Copy Markdown
Contributor

Addresses: https://edx-wiki.atlassian.net/browse/LMS-1167

compatibility.js is needed for the pdf viewer to work in IE as per https://github.com/mozilla/pdf.js. Just added the missing link to the template.

@frrrances, @brianhw please review.

@adampalay

Copy link
Copy Markdown
Contributor

This looks good to me 👍

@brianhw

brianhw commented Sep 26, 2013

Copy link
Copy Markdown
Contributor

👍 Seems pretty clearly stated.

@frrrances

Copy link
Copy Markdown
Contributor

👍

symbolist added a commit that referenced this pull request Sep 27, 2013
PdfViewer: Added missing link to compatibility.js
@symbolist
symbolist merged commit 1737984 into master Sep 27, 2013
@symbolist
symbolist deleted the usman/lms1167-pdfjs-update branch September 27, 2013 09:49
lgp171188 pushed a commit to open-craft/openedx-platform that referenced this pull request Sep 20, 2018
cocococosti pushed a commit to Pearson-Advance/edx-platform that referenced this pull request Aug 18, 2020
PE-785 - Adding custom course home feature
iloveagent57 pushed a commit that referenced this pull request Feb 26, 2024
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.

4 participants