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

Change :read_only field option to :writable #4472

Merged
merged 3 commits into from
Aug 5, 2024

Conversation

greg-rychlewski
Copy link
Member

This is based off our conversation in the insert_all update syntax PR. Our discussion brought to light that read_only is probably not granular enough.

This PR changes read_only to writable with allowed values :always, :insert, :never. :insert means it can only be created once but not modified once it's in the database. So it can't be used in on_conflict for upserts.

For reference this is the old PR: #4335. I used it to help me see where the changes needed to be.

@greg-rychlewski greg-rychlewski merged commit f01fa4a into elixir-ecto:master Aug 5, 2024
6 checks passed
@greg-rychlewski greg-rychlewski deleted the writable branch August 5, 2024 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants