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

docs: update field-level conditional settings #183

Merged
merged 1 commit into from
Oct 11, 2019
Merged

docs: update field-level conditional settings #183

merged 1 commit into from
Oct 11, 2019

Conversation

danirod
Copy link
Contributor

@danirod danirod commented Oct 9, 2019

The docs were not updated to reflect that both if and unless now expect three arguments for the conditional proc, even in the field settings. Passing two arguments as the previous version of the docs propose will yield a deprecation notice. This commit updates the docs so that it indicates how to use the three version argument.

Closes #179

The docs were not updated to reflect that both if and unless now
expect three arguments for the conditional proc, even in the field settings.
Passing two arguments as the previous version of the docs propose will yield
a deprecation notice. This commit updates the docs so that it indicates
how to use the three version argument.
@mcclayton mcclayton merged commit 6fa52e5 into procore-oss:master Oct 11, 2019
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.

Update documentation or fix deprecation notice for field-level conditionals?
3 participants