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.

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

Allow top-level groups to be stored in individual meta keys #836

Open
dlh01 opened this issue Apr 11, 2022 · 0 comments
Open

Allow top-level groups to be stored in individual meta keys #836

dlh01 opened this issue Apr 11, 2022 · 0 comments
Labels

Comments

@dlh01
Copy link
Member

dlh01 commented Apr 11, 2022

Repeatable groups were deliberately exempted from support for storing data in individual meta keys in #255:

The only way to support such a structure would be to save to meta keys like group_0_field, group_1_field, etc., which would not be sustainable and it would be difficult and inefficient to work with in code. In every situation where an ancestor's limit != 1, it's much better to use serialized data.

This makes sense for groups that have an ancestor, but it's not clear why the group at the top level of the field hierarchy would need to use the meta key structure outlined above. Instead, individual meta keys could store individual associative arrays.

My own use case for this enhancement is an "RSVP" group where each group represents a user-submitted RSVP. While storing all of the RSVPs in a single array works well enough, conceptually it would make a little more sense to me for each response to be stored in its own row.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant