Problem/Motivation
I've been reviewing this module and one thing that struck me is that I don't think I can see how a user can navigate to these diagrams from a content type. All the routes are in your own admin section and not connected to existing content type admin pages.
should it be?
Updated 2/24 I altered the title and the intent slightly so that a CM Document, if it exists for a bundle would link to the related entity reference diagram.
Updated 4/25
Acceptance Criteria
- When editing a content type or vocabulary, if you had permission to see the entity reference diagram you would see a tab that says "Entity Relationships" that would take you to /admin/reports/content-model/entity-diagram//?max_depth=2
- example: while here /admin/structure/types/manage/book you would see a tab that says "Entity Relationships" that would take you to /admin/reports/content-model/entity-diagram/node/book?max_depth=2
Issue fork content_model_documentation-3352712
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
swirtThis is a great thing to call out.
It is similar to this issue #3339245: Tabs for documents which would connect CM Documents to the entity UI for content entities and fields.
It is also similar to this one #3340574: Add Views relationship plugin that would allow links to CM document to show up in the field report.
Yours is a great suggestion to have the diagram available from other places.
Comment #3
swirtComment #4
swirtComment #5
nidhi27Hii @swirt,
Can you please guilde me on this so that I can understand the requirement.
Thanks!
Comment #6
swirtHi Nidhi27, sorry about that. I just added some Acceptance Criteria. Let me know if you have questions.
Comment #7
nidhi27Comment #9
nidhi27Hii,
I have created MR !41 to resolve this feature request. Let me know if any changes are needed.
Thanks!
Comment #10
nidhi27Hii @swirt,
I have tried to debug the issue but not able to understand why it disappears taxonomy only.
Comment #11
nidhi27Hii,
Issue is resolved. Now showing for taxonomy as well.
Thanks!
Comment #12
nidhi27Hi @swirt,
It was miss from phpcs bug. But here in this MR I have resolved. Thanks!
Comment #13
swirtFantastic work @nidhi27. This works better than I imagined it would. Very well done and thank you for this contribution.
Comment #14
swirtComment #15
nidhi27Comment #16
swirtThis went out with release 1.0.35