New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
load count.js from disqus.com over https #35
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
an entry on the changelog is missing; after merging this I can do a release for you.
@@ -22,7 +22,7 @@ def get_counter_js(self): | |||
|
|||
if short_name: | |||
result = """ | |||
<script type="text/javascript" async="async" src="http://{0}.disqus.com/count.js"> | |||
<script type="text/javascript" async="async" src="https://{0}.disqus.com/count.js"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
HTML5 specification urges authors to omit the attribute rather than provide a redundant MIME type; do you mind to remove the type
attribute also? instead we can use:
<script async src="https://{0}.disqus.com/count.js">
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/script
Thank you @hvelarde for the immediate review. Your requested changes are done. |
let me fix the Travis builds first. |
@philipsahli please update your branch so we can merge it. |
Travis build worked, Codacy Job still in state "Pending". Thank you in advance for the release and the great package! |
I made a new release for you: https://pypi.org/project/collective.disqus/2.2/ |
Fixes #34