HomePhabricator
Diffusion Comm 1ded84f26e78

[web] Fix default apps

Description

[web] Fix default apps

Summary:
I noticed that on web in default state we used defaultEnabledApps instead of defaultWebEnabledApps. This caused the calendar not to be visible to a just logged in user. This was also inconsitent with the fact that in the reduceEnabledApps reducer we set enabledApps to defaultWebEnabledApps on logout.
This was introduced in D9145 and MichaƂ says that it's a mistake.

Test Plan: Tested that the correct default is used now.

Reviewers: michal, ginsu

Reviewed By: michal, ginsu

Subscribers: ashoat, tomek

Differential Revision: https://phab.comm.dev/D10525

Details

Provenance
inkaAuthored on Jan 4 2024, 2:42 AM
Reviewer
michal
Differential Revision
D10525: [web] Fix default apps
Parents
rCOMM64ee95ff49d3: [identity] Use DB conversion helpers from comm-lib
Branches
Unknown
Tags
Unknown