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

Refactor ContentType/Field Reference #542

Open
toastercup opened this issue Nov 17, 2017 · 0 comments
Open

Refactor ContentType/Field Reference #542

toastercup opened this issue Nov 17, 2017 · 0 comments

Comments

@toastercup
Copy link
Member

Right now, a ContentType and its Fields are tied together by the Field's id. If a superadministrator is manipulating the ContentType via CLI, this is rather inconvenient. Instead, we should refactor the implementation so that Fields are tied to a ContentType via their unique, indexed name_id field. Let's research how we can do this performantly. If we're unable to do so, we should remove the name_id field and continue to reference by id, but create uniqueness validation logic scoped to a ContentType that normalizes the Field's name.

@toastercup toastercup changed the title Refactor ContentType->Field reference Refactor ContentType->Field reference Nov 17, 2017
@toastercup toastercup changed the title Refactor ContentType->Field reference Refactor ContentType/Field reference Nov 17, 2017
@toastercup toastercup changed the title Refactor ContentType/Field reference Refactor ContentType/Field Reference Nov 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant