- Issue created by @msnassar
- Issue was unassigned.
- Status changed to Needs review
almost 2 years ago 7:36am 28 January 2023 - 🇧🇪Belgium lobsterr
@msnassar I think, we have misunderstanding.
We have media finders plugins, which get us entities, where we can find medias, but also we need to know to which group we can assign these entities.
So, we have this hook groupmedia_entity_group, which allows to expose more groups to which we want attach medias.BTW, I think it is a good idea to have also group finder plugin. we can work on this later
Your plugin still can be useful to handle this case 📌 Handle paragraphs, when we add existing entity to a group Closed: duplicate .
Let me check one more time a bit later. I am still busy with another task currently - 🇧🇪Belgium msnassar
@LOBsTerr
I have already built group finder plugin for a website. I am planning to contribute it very soon...
It helps us to solve many cases. e.g.:
- Attach media to a group (that has a media field) when creating new group.
- Attach media using media library.
- Find group in pages other than the group page itself.
- .... - 🇧🇪Belgium lobsterr
@msnassar, yes, please, do it, but could you create a separate issue for it. It would much easier to review for me. Thanks
-
LOBsTerr →
committed 25f750bb on 8.x-2.x authored by
msnassar →
Issue #3337161 by msnassar, LOBsTerr: Use MediaFinder to add media in...
-
LOBsTerr →
committed 25f750bb on 8.x-2.x authored by
msnassar →
- Status changed to Fixed
almost 2 years ago 12:02am 10 February 2023 - 🇧🇪Belgium lobsterr
@msnassar Thank you for your contribution
I have checked your plugin it works as expected.
After double checking we don't need group finder hook, because we get necessary group ids from the entity to which the paragraph is attached. Plus it will be handled during the update! Automatically closed - issue fixed for 2 weeks with no activity.