Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[stable23] load dashboard widgets of enabled apps only #33156

Merged
merged 3 commits into from
Aug 1, 2022

Commits on Jul 27, 2022

  1. load widgets only of enabled apps

    - per design, all enabled apps have their registration run
    - limitations, e.g. enabled by group, are not considered in that state,
      because we do not have a session (and might need apps?)
    - before instantiation of widget it has to be checked whether the providing
      app is actually enabled for the logged in user.
    - a public interface is being changed, but it is not meant to be
      implemented or used outside of the core handling. Therefore save to
      backport.
    
    Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
    blizzz authored and PVince81 committed Jul 27, 2022
    Configuration menu
    Copy the full SHA
    8c87c0b View commit details
    Browse the repository at this point in the history
  2. avoid early DI of IAppManager

    - might break install
    
    Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
    blizzz authored and PVince81 committed Jul 27, 2022
    Configuration menu
    Copy the full SHA
    5c757f3 View commit details
    Browse the repository at this point in the history

Commits on Aug 1, 2022

  1. fix PHP 7.3 compat

    Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
    blizzz committed Aug 1, 2022
    Configuration menu
    Copy the full SHA
    f955fab View commit details
    Browse the repository at this point in the history