• Additional tests
• Correct how many SKUs to add at zero cost, beyond existing ones
• With doc
• Improve finding the entitleable object
• Adjust tests
• Avoid unintentional changes
• Fix phpstan issues
• Add testUpdateEntitlements
• Feedback
• Use counting on the sql server side

• Uncomment
• Rebase on current master

Needs rebasing for resources/sass/_variables.scss, then can be accepted I think.

• Feedback

• Simplify getting to the actual wallet
• Simplify getting to the wallet
• Fix test to meet expectations

• Set new and active outside the user id test
• Rebase on master

• Add testUpdateEntitlements

In D1297#14569, @machniak wrote:

Looking at the code in context of removing entitlements, I see we might not handle removing storage entitlements properly. We have to add some tests to tests/Feature/Controller/UsersTest::testUpdateEntitlements().

• Fix phpstan issues

• Avoid unintentional changes

• Additional tests
• Correct how many SKUs to add at zero cost, beyond existing ones
• With doc
• Improve finding the entitleable object
• Adjust tests

In D1297#14500, @machniak wrote:

So far so good. I have two stories to consider:

1. User is deleting and adding the same SKU every 13 days - got it for free.
2. (After 14 days of using his account) user raises quota, then in 14 days he lowers it - got the "rised quota period" for free.

This will need to be rebased on top of an accepted D1102.

• Use the $entitleable variable
• Use initialized variable

• The $id parameter is supposed to be a string
• A second proposal to transactions
• Update for D1102 review
• Resolve some comments for D1102
• Use constants
• Remove unintentional commenting out
• Add some tests

Rebase on current master;

Let us review the one single line item for the moment.

• Update for D1102 review

OK, I read no objections against the principles underpinning this implementation, but there are a few things that need fixing.

• The $id parameter is supposed to be a string
• A second proposal to transactions

I was being stupid and not using the admin URL ;-)

I'm not getting it. I apply this differential, and it doesn't yield any of what you describe.

• Fix merge problem
• New .gitignore public/js/ entries
• Also ignore tests/report
• Duplicate domain addition
• Correct namespace, correct wallet to use in function wallet() on Domain
• Increase resolution of desktop version
• ActiveSync <-> 2FA switched position
• Guarantee the ordering is as expected