sanity-io / sanity-plugin-media

Asset management plugin for Sanity
MIT License
158 stars 51 forks source link

Add `directories` to structure content #117

Open pascalwengerter opened 1 year ago

pascalwengerter commented 1 year ago

While I understand that tags and a filter get you very far, I can already see my editors asking for directories to do basic structuring of their media assets (say: "blogpost images", "landingpage images", "icons", ...)

AMarlonG commented 1 year ago

I second this ... I've worked in arts for years, and there the amount of image content and turn over is huge. Therefore folders is a must to keep track of e.g. artists, seasons, years, genres ...

patronstudio commented 1 year ago

+1

lundn commented 1 year ago

+1

didrikeilertsen commented 11 months ago

+1

michaelmannucci commented 9 months ago

I think this is sort of 101 for a media browser! No reply since May 2023 makes me sad lol

pascalwengerter commented 9 months ago

I think this is sort of 101 for a media browser! No reply since May 2023 makes me sad lol

I've since moved on from the employer where I introduced Sanity as CMS, but my impression throughout the last year was that Sanity has gone all-in on the AI hypetrain (being VC-funded, they probably figured that they don't have much choice). It's their product and their choice after all, but I had hoped that they'd more heavily invest into basic features because there are some great ideas at the foundation of the original product

Jamiewarb commented 7 months ago

I need this at work, so am going to spend a couple weeks building it into this plugin. I can submit a PR if @SimeonGriggs or @robinpyon would want functionality like this incorporated?

We could turn it on/off via a plugin setting, so that we maintain backwards compatibility, and so user's can disable the functionality if not needed.

I was thinking the following for functionality:

Will also need to think about how to display this if a user opens it from an Image field. Whether it can just look the same or would need to be adjusted at all

Future Extensions Other functionality we could consider later on is:

@SimeonGriggs / @robinpyon let me know if this would be a welcome addition to this plugin. Do let me know your thoughts on the feature set as well so it may better match your vision. The requirements above are just my initial thoughts

Or if you'd rather not integrate it for maintenance concerns, let me know too, and I'll see if I can stand-alone it as an add-on instead

AMarlonG commented 7 months ago

@Jamiewarb

I applaud this to the max. I am not much of a programmer for these types of issues, but if you need someone for testing, or any advice just shout out. I work as a project leader for web dev projects and can think thoughts, analyze and prioritize well.

I'm suprised Sanity isn't more interested in this themselves. Too much AI focus for them too ...? 😏

Jamiewarb commented 7 months ago

Hey @SimeonGriggs or @robinpyon, just wanted to follow this up to ask if you'd be interested in its addition?

SimeonGriggs commented 6 months ago

Hey @Jamiewarb let me bring it up internally and get someone to get back to you!

jtpetty commented 6 months ago

@Jamiewarb - I am an engineering manager at Sanity. It seems this is a long standing feature request that we have been unable to prioritize. I would welcome your contribution. Let's see what the code looks like in a PR and we can decide then if we feel it is too much of a maintenance burden but start with the intent that we merge it in.

michaelmannucci commented 6 months ago

@Jamiewarb did you end up building it and are asking if they want it? Or you have not built it yet?

Cuz I need it either way! 😅

Jamiewarb commented 6 months ago

Hey @jtpetty sounds great :) I'll aim to deliver something in the next 1-2 months, as the project I'm working on needs this for July. Hopefully it's closer to 1 than 2.

I'll aim to open a PR with an initial implementation, without some of the features I mentioned above, so that we can an early review and course correct if need be. Then I can add the rest.

And sorry @michaelmannucci it's going to be a little bit longer from my side

Jon-Myrstad commented 2 months ago

@Jamiewarb , did you get anywhere with this?

There's some clients I have on sanity that have requested this for ages and I've literally put Sanity further down in my "reccomendations" for choice of CMS because of this.

I personally don't have time to manage a PR like this myself, but if needed I'd love to contribute.

AMarlonG commented 6 days ago

@SimeonGriggs Did you bring it up internally, and if so, what's the status here?