We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
the current schema defines the condition values as properties:
helix-shared/src/schemas/conditions.schema.json
Line 45 in fa56cd9
which would lead to ugly trees for alternatives:
conditions: or: url.hostname=: www.adobe.com or: url.hostname=: www.adobe.io or: url.hostname=: www.adobe.net
of course, it could be written in a regexp, so maybe this is not really a problem
conditions: url.hostname~: www.adobe.(com|io|net)
an alternative could be to define the or, and nodes as arrays:
or
and
conditions: - or: - url.hostname=: www.adobe.com - url.hostname=: www.adobe.io - url.hostname=: www.adobe.net
or maybe both should be possible?
The text was updated successfully, but these errors were encountered:
or is an array. Your example should look like this:
conditions: or: - url.hostname=: www.adobe.com - url.hostname=: www.adobe.io - url.hostname=: www.adobe.net
Sorry, something went wrong.
@trieloff but then, those are wrong:
helix-shared/test/specs/configs/valid-conditions.yaml
Line 52 in 6eaad4f
No branches or pull requests
the current schema defines the condition values as properties:
helix-shared/src/schemas/conditions.schema.json
Line 45 in fa56cd9
which would lead to ugly trees for alternatives:
of course, it could be written in a regexp, so maybe this is not really a problem
an alternative could be to define the
or
,and
nodes as arrays:or maybe both should be possible?
The text was updated successfully, but these errors were encountered: