• Wallet discounts (by a separate table)
• Fix discount migration, use uuid as discount identifier
• Add simple discount tests
• Add discount test
• Display discounted prices in UI

• Wallet discounts (by a separate table)

• Improve logon form, fix tests, remove dummy method

I'd re-check on 1st of month.

Looks like a nice simple change.

In rK16b688d80237f2f24b9e01bfca7e692b2d0e6c1b#511210, @bohlender wrote:

in addition, the 2FA checkbox could use an explicit (as in not hidden behind the info button) hint
that this only works when the users also sets it up in roundcube,
as well as a hint that this will disable the usage of external clients for now

• Remove unused lang entries
• Fix phpstan errors

• Remove configured factors on entitlement delete, added tests

We'll move forward with a new table: discounts { id, discount, description, code (for vouchers), active } and wallets.discount_id.

I would do all these calculations on the server side. Probably returned with extended domains/users api requests, or maybe better via /api/v4/wallet/{wallet-id} (which does not exist yet).

• Add console script to set/get user status
• Add console command to set/get domain status
• Browser tests for status indicators
• Make user:status and domain:status commands available in non-production mode only
• CS and typos fixes

Reopen

Fix

CS fixes

@bohlender, there are two helper commands you can use to test the functionality.

$ php artisan user:status --del=imapReady
$ php artisan user:status --add=imapReady
$ php artisan domain:status --del=verified
$ php artisan domain:status --add=verified

• Add console script to set/get user status
• Add console command to set/get domain status
• Browser tests for status indicators

• RedirectUrl don't have to public

• Change webhook location to fix auth issue

• The proxy on 127.0.0.1 can be trusted
• Added APP_PUBLIC_URL setting

• Reset mollie_id when testing payment

• Set locale for Mollie to fix browser test

• Tests for payments controller
• Browser test for payment with Mollie

• Add browser tests, fix small issues

• Use Queue:: syntax, phpstan is now happy

• Make UserVerifyTest less fragile - wait up to 10 seconds

• Fix tests regressions

• Rebase
• Small refactoring regarding wallets/accounts
• Remove redundant code

Done in bdfb95825b525 [syncroton master].

Looks like the mentioned bug has been fixed in PHP 5.2.2, so I'm guessing we can remove the whole method. Oh, actually upstream did it here https://github.com/tine20/syncroton/commit/e847e680ba19d65aa5305d73313a5ea7af2387a7. So, I'm going to update Syncroton version we're using.

• Fix dispatching DomainDelete job - use ID instead of the Model object

• Use wallets to control access to UI elements

• Fixes regarding wallets

• Set domain/user STATUS_DELETED after removing from ldap
• Added more browser tests (that are failing at the moment)

I reviewed it again, I guess it's ready to be merged.

Yes, I just copied some code I already have had in Roundcube for future use. The main point was to have possibility to extend the browser object with some helper code, to simplify tests writing.

• Move fontawesome icons selection to a separate file

Looks good for now.