Resolve "Error when trying to install Drutopia Case Study"

cedewey commented

2018-10-18 17:03:27 +00:00

(Migrated from gitlab.com)

Closes #94

This adds the missing configuration for the Case Study content type. Also, the license vocab configuration had been mistakenly added to the Case Study feature. I moved this to its proper home in Creative License.

I don't know if I should by including the Config Snapshot provided file - config_sync.module.drutopia_case_study but it's being suggested through the Features UI so it's in there.

Closes #94 This adds the missing configuration for the Case Study content type. Also, the license vocab configuration had been mistakenly added to the Case Study feature. I moved this to its proper home in Creative License. I don't know if I should by including the Config Snapshot provided file - config_sync.module.drutopia_case_study but it's being suggested through the Features UI so it's in there.

mlncn commented

2018-10-18 20:04:46 +00:00

(Migrated from gitlab.com)

@cedewey that would be a bug with Features. Core to look/file? The snapshot should not ever be part of the install, optional, or any other sort of configuration. Not sure what happens if snapshot tries to snapshot itself but anyway it shouldn't be there.

mlncn commented

2018-10-18 20:05:57 +00:00

(Migrated from gitlab.com)

@cedewey Please remove this file; i don't see a straightforward way to throw it out when merging.

mlncn commented

2018-10-18 20:14:01 +00:00

(Migrated from gitlab.com)

@cedewey And are we missing any taxonomy term configuration between the two modules? Or is core.entity_view_display.taxonomy_term.license.default.yml it? No form view mode, or display view mode for the other vocabularies?

@cedewey And are we missing any taxonomy *term* configuration between the two modules? Or is `core.entity_view_display.taxonomy_term.license.default.yml` it? No form view mode, or display view mode for the other vocabularies?

cedewey commented

2018-10-18 20:51:43 +00:00

(Migrated from gitlab.com)

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://gitlab.com/agaric/sites/agaric-com/merge_requests/25/diffs?diff_id=25455529&start_sha=1ee97a93e0c9935a645520b240b2fa5859db2a7f#7c44e4b4744f83f9580f833e27757660b8d1a81f_1_0)

cedewey commented

2018-10-18 20:51:43 +00:00

(Migrated from gitlab.com)

added 1 commit

c493abe8 - Remove config snapshot file

Compare with previous version

added 1 commit <ul><li>c493abe8 - Remove config snapshot file</li></ul> [Compare with previous version](https://gitlab.com/agaric/sites/agaric-com/merge_requests/25/diffs?diff_id=25455529&start_sha=1ee97a93e0c9935a645520b240b2fa5859db2a7f)

cedewey commented

2018-10-18 20:52:07 +00:00

(Migrated from gitlab.com)

ok, I removed this.

cedewey commented

2018-10-18 20:52:08 +00:00

(Migrated from gitlab.com)

resolved all discussions

cedewey commented

2018-10-18 20:54:14 +00:00

(Migrated from gitlab.com)

@mlncn I checked and there isn't a view mode or display view mode to include for the other vocabularies.