acquia / df

Demo Framework - mirrored at https://git.drupal.org/project/df.git
https://www.drupal.org/project/df
18 stars 19 forks source link

Added config rewrite for the Video #77

Closed saltednut closed 6 years ago

saltednut commented 6 years ago

Media default display to remove title and author by default

kevinfunk commented 6 years ago

The displays are still showing. It is possible that the file is being overridden?

saltednut commented 6 years ago

Sorry, I should have mentioned you'll need to enable the module if testing with the df profile. I tried enabling it and actually saw an issue where we are installing core.entity_view_mode.media.media_browser in addition to Lightning installing it, which is weird to me? Anyway, I moved it into /rewrite - this may be an artifact of how the old system was set up before we had sub profiles.

saltednut commented 6 years ago

Ok actually... as you can see from the earlier failing tests, this was set up correctly to work with subprofiles of df. So in order to test this, rather than doing drush si df then drush en df_tools_media you'll want to do drush si dfs_dev and then check for the changes in configuration either by looking at /admin/structure/media/manage/video/display or simply inspecting it via drush drush cget core.entity_view_display.media.video.default