Launchpad itself

  1. Bug #837290
  2. Comment #1

Comment 1 for bug 837290

Revision history for this message
Nigel Jones (dev-nigelj) wrote :

I discussed this bug with 'gmb' (Graham Binns) on IRC and am doing a patch that will:

Alter lib/lp/app/javascript/subscribers/subscribers_list.js:SubscribersLoader to merge default values into 'this.config' as well as 'this'.
Add a unit test that reruns 3 of the resetNoSubscribers() tests via SubscribersLoader which will test that merging default values works correctly and produces expected output.

Merge proposal coming soon.