search

plasmicapp / plasmic

Visual builder for React. Build apps, websites, and content. Integrate with your codebase.
https://www.plasmic.app
MIT License
5.07k stars 416 forks source link

Contentstack package: Add limit and order to fetcher #54

Open Thomas-Wesley-BSA opened 2 years ago

vercel[bot] commented 2 years ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
contenful-example ✅ Ready (Inspect) Visit Preview Oct 19, 2022 at 4:13AM (UTC)
dynamic-multistep-forms ✅ Ready (Inspect) Visit Preview Oct 19, 2022 at 4:13AM (UTC)
plasmic ✅ Ready (Inspect) Visit Preview Oct 19, 2022 at 4:13AM (UTC)
plasmic-blog-example ❌ Failed (Inspect) Oct 19, 2022 at 4:13AM (UTC)
plasmic-blog-starter ❌ Failed (Inspect) Oct 19, 2022 at 4:13AM (UTC)
vercel-workflow ✅ Ready (Inspect) Visit Preview Oct 19, 2022 at 4:13AM (UTC)
vercel[bot] commented 2 years ago

@Thomas-Wesley-BSA is attempting to deploy a commit to the Plasmic Team on Vercel.

A member of the Team first needs to authorize it.

yang commented 2 years ago

Very cool!

Thomas-Wesley-BSA commented 1 year ago

More wanted functionality came up at work today - namely ability to choose which field to match on for the "Entry UID" prop, with the current behavior (UID) being the default.

We want to use dynamic paths to pull a single entry, and this will let us use url/pretty-custom-slug instead of url/4f7ds89a7gre

I've drafted the PR until said functionality is done, then I'll reopen it.

Thomas-Wesley-BSA commented 1 year ago

Done. It is only meant to work properly for string fields at the moment. It would be possible to fix this in the future, as we have access to the schema so there can be different behavior for string/number/boolean/etc... But to implement that properly the prop type will need to be dynamic, which it isn't at the moment (and I imagine would be a complicated change).