Skip to content

feat: payloadcms#1591

Open
z-zahi wants to merge 8 commits into
cloudflare:mainfrom
z-zahi:feat(payloadcms)
Open

feat: payloadcms#1591
z-zahi wants to merge 8 commits into
cloudflare:mainfrom
z-zahi:feat(payloadcms)

Conversation

@z-zahi

@z-zahi z-zahi commented May 25, 2026

Copy link
Copy Markdown

Payloadcms example working with vinext working with local sqlite, can be configured to use cloudflare d1 or other database using adapters

https://payloadcms.com/docs/database/overview

Screenshot 2026-05-25 at 6 54 06 PM

@james-elicx james-elicx left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

i would be interested in us getting the payload cms cloudflare d1 working as an ecosystem fixture.

any payload fixture should be based on their create-payload-app output and include their auto-generated tests that run in CI.

I pushed some changes to main...james/payload-cms to show what I mean.

It currently fails to run due to the file-type dep used in Payload.

@james-elicx james-elicx marked this pull request as draft May 27, 2026 08:37
@z-zahi z-zahi marked this pull request as ready for review May 27, 2026 08:49
@z-zahi

z-zahi commented May 27, 2026

Copy link
Copy Markdown
Author

I made the changes against main...james/payload-cms, it now works with d1 and r2, fixed the file-type and @payloadcms/ui/elements/HTMLDiff/diff temporaraly, unfortunately vinext is not detecting exports correclty so I added aliases, the tests pass and the app works very smoothly, I'd like to know your opinion and if there's something missing @james-elicx

@james-elicx

Copy link
Copy Markdown
Member

Would you be able to do it in separate commits so we can track exactly what needs to change from the Payload-created template for their apps to work? It's not clear in a single commit how much things change from their template.

unfortunately vinext is not detecting exports correclty so I added aliases

This sounds like a potential bug or issue that we should ideally look at fixing

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants