-
Notifications
You must be signed in to change notification settings - Fork 0
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
UX Fixes #74
Conversation
Pull Request Test Coverage Report for Build 364
💛 - Coveralls |
@@ -47,6 +47,8 @@ class IIftttTriggerAction(Interface): | |||
u'Give the name of IFTTT event which you want to trigger' | |||
), | |||
required=True, | |||
# BUG - add one word contraint to this and to all ifttt_triggers | |||
# constraints=validate_ifttt_event, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Shriyanshagro Folloe https://docs.plone.org/develop/addons/schema-driven-forms/customising-form-behaviour/validation.html#constraints to add "constaint" to schema
constraint is a function that receives value to checked as its only argument (it knows nothing about the context of the value)
if values is ok it returs nothing
if value is not ok, it raises exception (see the exception class from the linked doc)
@Shriyanshagro I removed SelectWidget from this branch into #75 |
This Pr includes major UX fixes