Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Deprecate ChangeSet.fields and .fields= #826

Open
no-reply opened this issue Apr 29, 2020 · 0 comments
Open

Deprecate ChangeSet.fields and .fields= #826

no-reply opened this issue Apr 29, 2020 · 0 comments

Comments

@no-reply
Copy link
Contributor

These methods provide a partial, asymmetrical interface to .property and .definitions. their use probably isn't reliable in general, and they don't seem to be in widespread use.

the use case they are meant to meet is bulk definition of properties. the easy alternative is my_field_list.each { |field| property(field) }.

we should deprecate this in favor of applications defining properties explicitly.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
Status: Backlog
Development

No branches or pull requests

1 participant