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

fix: conflicting where inputs #1734

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

davenewza
Copy link
Contributor

@davenewza davenewza commented Feb 21, 2025

There is no need to restrict a field which is filtered by inputs and in @where. See example schema below where this case would be valid. This PR relaxes that condition, but we still validate against using write values and @set on the same field.

model Order {
    fields {
	quantity Number
	price Decimal
	category Text
	status Status
    }
    actions {
	list listOrders(status?, category?) {
            @where(order.status != Status.Cancelled)
	}
    }
}

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.

1 participant