Homec4science

Fix issue where "phabricator" disqus account is used by default and not…

Authored by Cam Spiers <camspiers@gmail.com> on Jun 26 2013, 01:28.

Description

Fix issue where "phabricator" disqus account is used by default and not overridable

Summary: Currently setting "disqus.shortname" via config isn't actually used in the instantiation of disqus. This fix uses the shortname configured.

Test Plan: Open a phame post with disqus enabled and a shortname other than "phabricator" specified.

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, Korvin

Differential Revision: https://secure.phabricator.com/D6310

Details

Committed
epriestley <git@epriestley.com>Jun 26 2013, 01:28
Pushed
aubortJan 31 2017, 17:16
Parents
rPH78311f758d59: Make ignored setup issues grey
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH0495eb15862e: Fix issue where "phabricator" disqus account is used by default and not… (authored by Cam Spiers <camspiers@gmail.com>).Jun 26 2013, 01:28