Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: folder enabled collections #10030

Draft
wants to merge 33 commits into
base: main
Choose a base branch
from
Draft

feat: folder enabled collections #10030

wants to merge 33 commits into from
+6,167 −120

Conversation

JarrodMFlesch
Copy link
Contributor

No description provided.

@JarrodMFlesch JarrodMFlesch changed the title Feat/folders feat: folder enabled collections Dec 17, 2024
@himanshu-ntml
Copy link

himanshu-ntml commented Dec 23, 2024 •
edited
Loading

Excited for the folders! Thank you so much, everyone. You all are rocking it!

rilrom
rilrom reviewed Jan 11, 2025
View reviewed changes
packages/next/src/views/Document/renderDocumentSlots.tsx Outdated
@@ -115,6 +115,14 @@ export const renderDocumentSlots: (args: {
}
}

if (collectionConfig.upload && collectionConfig?.admin?.components?.edit?.Upload) {
Copy link
Contributor

@rilrom rilrom Jan 11, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

collectionConfig.upload throws a type error for globals. It needs to be collectionConfig?.upload.

I discovered this while testing #9925.

@RobinSCU
Copy link

RobinSCU commented Feb 7, 2025

Hey @JarrodMFlesch,
just wanted ask if there is possibility the Beta test that feature. We are currently switch from an older CMS to Payload an one big Topic is the Media Management. I think we have something like 5000 images and because of that folders would be amazing 😄

@nachomglz
Copy link
Contributor

hey, is there an ETA for "non-enterprise" users?

@RobinSCU
Copy link

Lets go @JarrodMFlesch!! i believe in you :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rilrom rilrom rilrom left review comments

@denolfe denolfe Awaiting requested review from denolfe denolfe will be requested when the pull request is marked ready for review denolfe is a code owner

@jmikrut jmikrut Awaiting requested review from jmikrut jmikrut will be requested when the pull request is marked ready for review jmikrut is a code owner

@DanRibbens DanRibbens Awaiting requested review from DanRibbens DanRibbens will be requested when the pull request is marked ready for review DanRibbens is a code owner

@AlessioGr AlessioGr Awaiting requested review from AlessioGr AlessioGr will be requested when the pull request is marked ready for review AlessioGr is a code owner

@jacobsfletch jacobsfletch Awaiting requested review from jacobsfletch jacobsfletch will be requested when the pull request is marked ready for review jacobsfletch is a code owner

Assignees
No one assigned
Labels
created-by: Payload team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@JarrodMFlesch @himanshu-ntml @RobinSCU @nachomglz @rilrom