extension doesn't update subscription

[sentientwaffle]

I had a (staging) extension stuck in a state with its user field

{"id":"ckkh3l0mw1kjj0798ch748p5i","fullName":null,"customerId":"cus_IqKQzSxuDjooG7","subscription":{"active":false,"endDate":"2021-08-28T00:00:00.000Z","trialEndDate":"2021-07-28T16:55:05.155Z"},"currencyPreferences":null}

(note the active: false despite a future endDate).

cc: @njlie @sublimator

[sublimator]

Strange, what branch you were using for staging ?

[sentientwaffle]

The extension was the batch payments branch, but I don't think that was related -- it doesn't touch anything related to subscriptions afaik.

The server was on main.

[sublimator]

The only way it could be related is if the batch payments branch doesn't include latest changes circa user tracking from main. I think it does (99%)

[sentientwaffle]

git log in the build:

* 676d400a sentientwaffle: fix(extension): dont "overpay" via partial payment -  (HEAD -> dj-payment-scheduler-post, origin/dj-payment-scheduler-post)(6 weeks ago)
* a96d44fa Nicholas Dudfield: chore(coil-extension): log rate - (6 weeks ago)
* 0523103a Nicholas Dudfield: chore(coil-extension): log rate - (6 weeks ago)
*   032f34a7 Nicholas Dudfield: Merge remote-tracking branch 'origin/main' into dj-payment-scheduler-post - (6 weeks ago)
|\
| * e5226afa Nicholas Dudfield: chore(coil-extension): honour WEXT_MANIFEST_VERSION_NAME, closes #2049 (#2052) - (6 weeks ago)
| * b9460488 Nicholas Dudfield: docs(coil-extension): note issue with node > 14 (#2051) - (6 weeks ago)
* |   5fd6fa06 Nicholas Dudfield: Merge remote-tracking branch 'origin/dj-payment-scheduler-post' into dj-payment-scheduler-post - (6 weeks ago)
|\ \
| * | 781287e5 sentientwaffle: chore(polyfill-utils): disable batching (temp) - (7 weeks ago)
* | |   ece50996 Nicholas Dudfield: Merge remote-tracking branch 'origin/main' into dj-payment-scheduler-post - (6 weeks ago)
|\ \ \
| |/ /
|/| /
| |/
| * 74c11182 renovate[bot]: chore(deps): update dependency html-webpack-plugin to v5.3.2 (#2047) - (6 weeks ago)
...

So it looks should.