Problem/Motivation
When I try to add new Media entity, I need to do that by typing /media/add in address bar.
There is no link in admin menu or none of the pages for adding new Media entity. Also, there is no link and page when media entities are listed.
Proposed resolution
It will be good to create page with table list of Media entities with links view/edit/delete. Local task can be created under the /config/content menu item with path /admin/content/media.
Add media link also can be placed on that page like local action.
Comment | File | Size | Author |
---|---|---|---|
#5 | media-entity-menu-link-adding-media-2622190-5.patch | 357 bytes | mbovan |
| |||
#5 | add_media_button.png | 35.1 KB | mbovan |
#2 | media-entity-menu-link-adding-media-2622190-2.patch | 21.54 KB | woprrr |
Comments
Comment #2
woprrr CreditAttribution: woprrr as a volunteer commentedHi @euphoric_mv
I work actualy in this issue, @see PR correspond : https://github.com/drupal-media/media_entity/pull/58
This is the patch ATM he doesn't work fine with MediaTestUi Test coverage. Why ? Views need a schema to declare added fields for "name" field. ATM I don't really understand why when we add an media views in config/optional this test crash ... (Probably it's an other bug).
I investigate !!
Comment #3
woprrr CreditAttribution: woprrr as a volunteer commentedComment #4
slashrsm CreditAttribution: slashrsm as a volunteer commentedMain focus of this issue is to put link to /media/add somewhere in the admin section of the side. Pull request in #2 is also adding a listing view. I am thinking that we should probably split that and stick to the main focus of this issue. we can add a listing page in a separate issue.
If we take this approach we have to decide where to put that link. Since there is no media listing page at the moment we have to find some other place. I was thinking that we could add another action to either content listing (next to "Add content" action) or to the file listing page.
Comment #5
mbovan CreditAttribution: mbovan at MD Systems GmbH commentedI added an Add media button to /admin/content. Wanted to add tests, but as there are some test methods commented-out, I didn't find the right place for it...
Comment #6
Lukas von BlarerThe button looks good to me.
Which tests are you talking about? Maybe we should test whether the button is present? But since we are going to move that to the file listing anyhow... Should we add tests for this in that separate issue?
Comment #7
slashrsm CreditAttribution: slashrsm at MD Systems GmbH commentedCommitted. Thanks!
Created listing follow-up #2626064: Consider adding default media listing view.
Comment #9
slashrsm CreditAttribution: slashrsm at MD Systems GmbH commented