* Fix: Invoices collected array length comparison
Update the code with the correct condition to respect the intent of the previous implementation ("exit with non zero code when no invoicess were processed").
See 5476f39984
However, I'm not sure if erroring when no invoices are collected is actually what we want to do.
* Wrap `collect_paypal_past_due_invoice` script and export the function
* Fixup typo `accoutns`
* Log invoices collection data before throwing
* Add note: `handleAPIError` is silencing the errors
* Create a test on `collect_paypal_past_due_invoice`
* Replace `console.log` by `@overleaf/logger` (bunyan)
Our `console.warn` show up as Errors (in red) in GCP. For example the following is an error in GCP:
```
Errors in attemptInvoiceCollection with id=2693634 OError: Recurly API returned with status code: 400
```
https://github.com/overleaf/internal/blob/5476f39/services/web/scripts/recurly/collect_paypal_past_due_invoice.js#L9
---
Does it correctly set the levels as warnings if we use `@overleaf/logger`
GitOrigin-RevId: 37c8bdf4afd8cef4706700aafb44480ec8966a74
* Add `with-split-tests` util for storybook
It wraps storybook in a SplitTest context provider with configurable variant values. And it adds controls for them in the storybook UI
* Define `splitTestsArgTypes` in `withSplitTests`
* Revert "Add `with-split-tests` util for storybook"
This reverts commit 3ff351ac
* Edit stories: remove SplitTestProvider and add more digits to amounts
* Create `addon-split-tests` to manage split-tests from Storybook
* Add a `splitTests` parameter to define individual Stories relevant split-tests
* Revert "Add a `splitTests` parameter to define individual Stories relevant split-tests"
This reverts commit 6cbf62729a3d682d695c36e11a86bd576cb7aec1.
* Revert "Create `addon-split-tests` to manage split-tests from Storybook"
This reverts commit d0734a1773c96860e82d81fd62fc9034e2b6e41c.
* Revert "Revert "Add `with-split-tests` util for storybook""
This reverts commit 407fc39b
* Improve types of price-summary.stories.tsx
Co-authored-by: Alf Eaton <alf.eaton@overleaf.com>
---------
Co-authored-by: Alf Eaton <alf.eaton@overleaf.com>
GitOrigin-RevId: bcf3d9f317590a00fb633851ef36146eb800fcfc
* [web] Show confirmation modal when reverting a file
* only show Revert file in ui even if deleted
* format:fix
* using the same message for every revert
* replace "document" with "file" in confirm dialog
* remove "pull-left" class
* fix split test name
GitOrigin-RevId: ab5937b9a0570cd1fdbe5d71eda0a16ae539dc66
* Revert "Merge pull request #14143 from overleaf/jlm-unsampled-compiles"
This reverts commit 2457977d61756be52db506d7c84531b22a26abe0, reversing
changes made to 3c452b3943c453707aac90d38e7b5b2ea9df0df4.
* Re-add export of `getPercentile` in SplitTestHandler.js
Was previously removed in aa8c0aadec
GitOrigin-RevId: 42c48829e7246a6bdb9dbb2d075638c1eafdf2f0
* Load correct JS for the active Bootstrap version
* Tidy up bootstrapVersion declaration
* Add Bootstrap JS to website redesign layout
* FIx error on interstitial subscriptions page
* Remove unnecessary import of jQuery and Bootstrap
* Use global entrypointScripts in bootstrap-js mixin
GitOrigin-RevId: 6b1977354a72dc69008fc0d2e3baec2f28d97f6b
* Fix split-tests loading in React component: use `useSplitTestContext` instead of `getSplitTestVariant`
* Replace use of `isSplitTestEnabled` by `useSplitTestContext`
* Add SplitTestProvider to roots, and fix tests
* Create `useFeatureFlag` hook
* Use `useFeatureFlag` where applicable
GitOrigin-RevId: 9ff7bb3975d50bc4d07d74d93c482d56dc96f615
* [web] "back to editor" button when history is opened
* rename to shouldReopenChat
* move to separate component
* show online users widget
* using MaterialIcon
* import useState directly
* fix formatting
GitOrigin-RevId: c37432f16518ef83510c48d90722e74b228b5ab1
