• Hardcode copyright in the footer

• Add real favicon.ico

• Rollback transactions on exceptions
• Performance optimization

Partially implemented in https://git.kolab.org/D1372 and we decided to not implement the costs overview table. So, I think this differential could be abandoned.

• Fix History tab after rabease

• Use YYYY-MM format in receipt filename
• Rebase

• Small code improvements

• Use YYYY-MM format in receipt filename

@vanmeeuwen, I rebased, fixed obvious bugs and refactored the code so now we have Wallet::balanceLastsUntil() that we need tests for in tests/Feature/WalletTest::testBalanceLastsUntil().

• Fix error due to missing use statements
• Create Wallet::balanceLastsUntil()

This learned us that an exception does not rollback uncommitted transactions. Here's a general fix for this issue:

--- a/src/app/Exceptions/Handler.php
+++ b/src/app/Exceptions/Handler.php
@@ -5,6 +5,7 @@ namespace App\Exceptions;
 use Exception;
 use Illuminate\Auth\AuthenticationException;
 use Illuminate\Foundation\Exceptions\Handler as ExceptionHandler;
+use Illuminate\Support\Facades\DB;

My comments to src/app/Domain.php were not addressed, but it's not critical.

• Render fake receipt with VAT

• Top-up -> Payment

The spec. says Body is optional. https://docs.microsoft.com/en-us/openspecs/exchange_server_protocols/ms-ascal/70161e94-3b50-4d20-8c4d-e3ca608f9dc8

• Fix typo

• Fix Stripe auto-payment charge
• Fix Mollie auto-payment
• Add more tests

• Update transaction log with payment description
• Add wallet:mandate command to view/enable/disable auto-payment
• Disabled mandate handling, test and fixed some flaky tests

• Use transaction log

• Add some payment notification jobs/emails
• More on auto-top-ups and disabling mandates
• Added tests for two jobs
• Tests for Mollie wallet top-ups
• Rename templates
• Fix parse error
• Stripe: Webhook fixes, more tests
• Add todo
• More webhook tests, send notifications for recurring payments only
• Disable auto-payment mandate on failed payment
• Add payment success/failure notification emails with tests
• Payment mandate disabled notification
• Fix typos
• Small improvements
• Improve wording
• Add fakeRender() for all templates

I think we'll need this code as a method on the Wallet object, as it will be needed also for UI. And we need a test. I don't understand the code, so I might get back to it later.

Rebased

Tests look good.

One last change request. Is it finished?

• More webhook tests, send notifications for recurring payments only
• Disable auto-payment mandate on failed payment
• Add payment success/failure notification emails with tests
• Payment mandate disabled notification
• Fix typos
• Small improvements
• Improve wording

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().