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

(907) Allow fields with copy code to be configured #9953

Conversation

pezholio
Copy link
Contributor

@pezholio pezholio commented Feb 19, 2025

Trello card: https://trello.com/c/aSyUk0AR/907-allow-fields-with-copy-code-to-be-configured

This adds a new configuration file that allows us to configure which fields can have a copy code associated (defaults to all fields). This config file will also be useful for us to define fun things like custom fields and ordering of fields too.

Screenshot

image

This will allow us to set config in one place for particular schemas
and their field(s)
This will allow us to specify what fields are embeddable for a schema
or subschema. If blank, it defaults to all fields.
This will prevent having to pass a schema object around all the time
@pezholio pezholio force-pushed the content-modelling/907-allow-fields-with-copy-code-to-be-configured branch from 0336c55 to 14fd6fb Compare February 19, 2025 16:13
@pezholio pezholio changed the title Content modelling/907 allow fields with copy code to be configured (907) Allow fields with copy code to be configured Feb 19, 2025
@pezholio pezholio marked this pull request as ready for review February 19, 2025 16:19
@pezholio pezholio requested a review from Harriethw February 19, 2025 16:22
Copy link
Contributor

@Harriethw Harriethw left a comment

Choose a reason for hiding this comment

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

👍

@pezholio pezholio merged commit e345da6 into main Feb 19, 2025
19 checks passed
@pezholio pezholio deleted the content-modelling/907-allow-fields-with-copy-code-to-be-configured branch February 19, 2025 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants