Skip to content
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

Updated tags to use new Disqus format. Refs #75 #76

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

petersanchez
Copy link

Added variable mapping for backwards compatibility. This is to address issue #75

In my testing it works as expected (which 0.5 was no longer working for me - in dev environments)

Mostly related to the updated tag format for counts and showing comments. SSO didn't change. Dev tags are no longer needed / supported.

@SalahAdDin
Copy link

👍

@SalahAdDin
Copy link

@petersanchez, did you update your fork for work with API 3.0?

@petersanchez
Copy link
Author

I don't remember honestly. This was a year ago. So I did it with the latest version of the API at that time. It's been working for us all this time.

@SalahAdDin
Copy link

@petersanchez And you deleted your fork, isn't it?

@petersanchez
Copy link
Author

@SalahAdDin Sorry, I moved it to here: https://github.com/netlandish/django-disqus/

@SalahAdDin
Copy link

@petersanchez Well, Github has notified me so late, hahaha, thanks.

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