Fix: Nested document data

pemedia / payload-visual-editor

Payload CMS plugin which provides a visual live editor directly in the Admin UI. Works for collections and globals. Compatible with any kind of JS/TS based frontend technology.

MIT License

209 stars 10 forks source link

Fix: Nested document data #6

Closed DennisSnijder closed 1 year ago

DennisSnijder commented 1 year ago

Context

At the moment the getAllFields function only check for a depth of 1. On my payload setup I use both Tabs and Rows resulting in a field depth of 2. This resolved into those fields not being sent to the preview page.

What has been done

Made the getAllFields recursive to resolve all fields.
Filtered out UI fields when resolving fields
Refactored the function to return early and throw an error (with a report link in the console) when an unrecognised field appears.

DennisSnijder commented 1 year ago

@dkirchhof @jpeiniger ready for review 😄

dkirchhof commented 1 year ago

Great, thank you for your contribution. Will be merged. Later I will update the npm package.