Skip to content

Conversation

@apetraru-uipath
Copy link
Contributor

Added compatibility for guardrails entitlements so that tests are passing.

"""Mock guardrail evaluation - always passes validation."""
return GuardrailValidationResult(validation_passed=True, reason="")
return GuardrailValidationResult(
validation_passed=True, reason="", result="passed", details=""
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't need details param. It is not in the GuardrailValidationResult object.
We should also remove it from uipath-python: https://github.com/UiPath/uipath-python/pull/1092/files#diff-14296796b856028cfcc68bafe25ca88f16c97c95e84771d7e900b27bd4aa46deR83

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've discussed this topic with @ctiliescuuipath and we intent to deprecate fields: reason & validation_passed.
In the end, we will keep only fields result, details.

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