Page MenuHomePhabricator

Add JavaScript checks so that AFT5 is only displayed for our small test sample
Closed, ResolvedPublic


Add JavaScript checks in AFT5 Feedback Forms to make sure that AFT5 is only displayed for articles that are in the categories for our small test sample.

In the past few days following launch of the Option 6 feedback form, we had lots of complaints that AFT was appearing in lots of other pages, and Dario reports that we generated a lot more traffic (a third of all AFT4), which is a pretty serious issue. Also, there were so many calls to one of our APIs that we had to shut it down.

It appears that the new JS for Option 6 didn't check categories and that old HTML from cache (which loads ext.articleFeedbackv5.startup unconditionally) caused this to happen.

So probably the best approach is to duplicate the check in JS for now until the cached HTML goes away, which could take another 30 days.

This commit from Reha puts the js check back:

One more question: should we change the name of the JS file, so that it would force people to reload that JS, rather than still use the old JS?

Version: unspecified
Severity: critical



Event Timeline

bzimport raised the priority of this task from to Unbreak Now!.Nov 22 2014, 12:22 AM
bzimport added a project: ArticleFeedbackv5.
bzimport set Reference to bz37414.