Differential D2659

Tenant settings
ClosedPublic
Actions

Authored by machniak on Tue, Jul 13, 11:04 AM.

Details

Reviewers

vanmeeuwen

Group Reviewers

Restricted Project

Commits

rKb9b80fe18cff: Tenant settings

Summary

Add per-tenant settings and support running jobs for another tenant. Also removed caching from the SettingsTrait.

Test Plan

./phpunit

Diff Detail

Repository

rK kolab

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

machniak requested review of this revision.Tue, Jul 13, 11:04 AM

machniak created this revision.

Harbormaster completed remote builds in B35614: Diff 7531.Tue, Jul 13, 11:04 AM

Add getSettings() method, and use it as a optimization tool

Harbormaster completed remote builds in B35632: Diff 7537.Thu, Jul 15, 10:07 AM

Fix withObjectTenantContext macro
- It's result should not depend on the current user role
- It was throwing exceptions when working with an unauth user (e.g. /meet endpoints)
- Fixed some tests

Harbormaster completed remote builds in B35725: Diff 7576.Wed, Jul 28, 2:11 PM

vanmeeuwen accepted this revision.Fri, Jul 30, 7:46 AM

This revision is now accepted and ready to land.Fri, Jul 30, 7:46 AM

Closed by commit rKb9b80fe18cff: Tenant settings (authored by machniak, committed by vanmeeuwen). · Explain WhyFri, Jul 30, 7:53 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

		Path
M		src/app/Backends/LDAP.php (8 lines)
A	M	src/app/Console/Commands/Scalpel/TenantSetting/CreateCommand.php (14 lines)
A	M	src/app/Console/Commands/Scalpel/TenantSetting/ReadCommand.php (15 lines)
A	M	src/app/Console/Commands/Scalpel/TenantSetting/UpdateCommand.php (14 lines)
A	M	src/app/Console/Commands/Tenant/ListSettingsCommand.php (43 lines)
M		src/app/Documents/Receipt.php (5 lines)
M		src/app/Http/Controllers/API/V4/OpenViduController.php (7 lines)
M		src/app/Http/Controllers/API/V4/PaymentsController.php (13 lines)
M		src/app/Http/Controllers/API/V4/WalletsController.php (2 lines)
M		src/app/Jobs/PasswordResetEmail.php (7 lines)
M		src/app/Jobs/PaymentEmail.php (30 lines)
M		src/app/Jobs/PaymentMandateDisabledEmail.php (29 lines)
M		src/app/Jobs/SignupInvitationEmail.php (7 lines)
M		src/app/Jobs/SignupVerificationEmail.php (7 lines)
M		src/app/Jobs/WalletCheck.php (49 lines)
M		src/app/Mail/Helper.php (69 lines)
M		src/app/Mail/NegativeBalance.php (12 lines)
M		src/app/Mail/NegativeBalanceBeforeDelete.php (11 lines)
M		src/app/Mail/NegativeBalanceReminder.php (11 lines)
M		src/app/Mail/NegativeBalanceSuspended.php (11 lines)
M		src/app/Mail/PasswordReset.php (11 lines)
M		src/app/Mail/PaymentFailure.php (14 lines)
M		src/app/Mail/PaymentMandateDisabled.php (14 lines)
M		src/app/Mail/PaymentSuccess.php (14 lines)
M		src/app/Mail/SignupInvitation.php (9 lines)
M		src/app/Mail/SignupVerification.php (2 lines)
M		src/app/Mail/SuspendedDebtor.php (19 lines)
M		src/app/Providers/AppServiceProvider.php (15 lines)
M		src/app/Providers/Payment/Mollie.php (7 lines)
M		src/app/Providers/PaymentProvider.php (6 lines)
M		src/app/Tenant.php (51 lines)
A	M	src/app/TenantSetting.php (30 lines)
M		src/app/Traits/SettingsTrait.php (76 lines)
M		src/app/User.php (7 lines)
M		src/app/Utils.php (10 lines)
A	M	src/database/migrations/2021_07_12_100000_create_tenant_settings_table.php (45 lines)
M		src/resources/lang/en/app.php (2 lines)
M		src/tests/Browser/Meet/RoomControlsTest.php (2 lines)
M		src/tests/Browser/Meet/RoomSetupTest.php (4 lines)
A	M	src/tests/Feature/Console/Scalpel/TenantSetting/CreateCommandTest.php (44 lines)
A	M	src/tests/Feature/Console/Scalpel/TenantSetting/UpdateCommandTest.php (47 lines)
A	M	src/tests/Feature/Console/Tenant/ListSettingsTest.php (59 lines)
M		src/tests/Feature/Jobs/PasswordResetEmailTest.php (6 lines)
M		src/tests/Feature/TenantTest.php (25 lines)
M		src/tests/Feature/UserTest.php (12 lines)
M		src/tests/MailInterceptTrait.php (2 lines)
M		src/tests/Unit/Mail/HelperTest.php (70 lines)
M		src/tests/Unit/Mail/NegativeBalanceBeforeDeleteTest.php (63 lines)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	7531	4eea276		Tue, Jul 13, 11:04 AM	★	★
Diff 2	7537	4eea276	- Add getSettings() method, and use it as a optimization tool	Thu, Jul 15, 10:07 AM	★	★
Diff 3	7576	4eea276	- Fix withObjectTenantContext macro	Wed, Jul 28, 2:10 PM	★	★
Diff 4	7612	7c016da	rKb9b80fe18cfff07100990a2c21c76f1ee6e95588	Fri, Jul 30, 7:47 AM	★	★