Dashboard config file dependencies do not include the dashboard module

Created on 17 August 2024, 5 months ago
Updated 30 August 2024, 5 months ago

Problem/Motivation

When you create a dashboard in the UI, and export the configuration file, the dependencies section does not include the dashboard module.

Steps to reproduce

Create a dashboard
Create a Dashboard text block
Save
Export config
Verify in the config file the dependencies include layout_builder but not dashboard.

Proposed resolution

The dashboard module should be included as a dependency.

Remaining tasks

Discussing, Deciding, Doing.

User interface changes

TBD

API changes

TBD

Data model changes

TBD

πŸ› Bug report
Status

Closed: works as designed

Version

2.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States thejimbirch Cape Cod, Massachusetts

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @thejimbirch
  • Status changed to Postponed: needs info 5 months ago
  • πŸ‡ͺπŸ‡ΈSpain plopesc Valladolid

    Hello Jim!

    Thank you for flagging this possible problematic scenario.

    Checked other config entities provided by core modules, like views or blocks, and they don't have an explicit dependency to the module that provides the entity type.

    From my point of view, it should not be necessary to add the explicit dependency on dashboard module in dashboard config entities.

    However, you might have found an scenario where this is generating any issue, it should be addressed.

    If that's the case, please report the steps to reproduce the issue and we could determine the best way to solve it.

  • Status changed to Closed: works as designed 5 months ago
  • πŸ‡ΊπŸ‡ΈUnited States thejimbirch Cape Cod, Massachusetts

    I asked about this in the Recipes channel of the Drupal slack, and config expert bircher stated:

    the first part ie the module name is an implicit dependency

    that is the module where the config entity is defined, it will always implicitly depend on that module

    https://drupal.slack.com/archives/C2THUBAVA/p1724847485173839?thread_ts=...

    So the name itself is an even stricter check on dependencies and it is not needed in the file.

    Thanks for your attention, I will close this as works as designed.

Production build 0.71.5 2024