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

Workflow Forms no longer respect the "no validation" button type #4713

Closed
chrismcgrath opened this issue Jun 10, 2021 · 1 comment
Closed
Labels
Status: Confirmed It's clear what the subject of the issue is about, and what the resolution should be. Topic: Workflows Type: Enhancement Feature requests. x-Fixed in v12.7
Milestone

Comments

@chrismcgrath
Copy link
Contributor

chrismcgrath commented Jun 10, 2021

Description

Selecting the Primary (no validation) Button Type on a Workflow Form does not prevent the form from requiring a value in a required field. Noticed first in v12.2 but confirmed it's happening in 12.4 on rock.rocksolidchurchdemo.com.

I created this Test Button Validation workflow at https://rock.rocksolidchurchdemo.com/page/136?workflowTypeId=24 but if it's been deleted or you want to test on another install, here is the workflow, just remove .txt from the filename and import.
Test Button Validation_202106101146.json.txt

Steps to Reproduce

  1. Create a Workflow with an attribute and an entry form.
  2. Make the attribute editable on the entry form.
  3. Make two buttons on the form, a Submit button with Primary button type and a Cancel button with Primary (no validation) button type.
  4. Run the workflow
  5. Click the cancel button

Expected behavior:

The form should submit/cancel successfully.

Actual behavior:

The form says that the required field is required.

Interestingly, the validation displays differently when using the Submit button or the Cancel button.

Submit button:

image

Cancel button:

image

Versions

  • Rock Version: Rock McKinley 12.2 (1.12.2.2), Rock McKinley 12.4 (1.12.4.1)
  • Client Culture Setting: en-US
@cwomack cwomack added the Status: Confirmed It's clear what the subject of the issue is about, and what the resolution should be. label Jun 11, 2021
@nairdo
Copy link
Member

nairdo commented Jun 16, 2021

@chrismcgrath I was able to dig into this one and see what was going on. It turns out the core/stock "Cancel" button is special and has extra stuff that happens behind the scenes -- namely that it is specifically tuned to disable validation when it is used in the workflow form. (To put it another way: there is no visible/administrative surface to control whether a button should 'cause validation' in the Button HTML defined type.)

I also reviewed this with the design team to conclude that this is not actually a bug, but I would understand if you decided you wanted to open a feature request over at http://community.rockrms.com/Ideas to perhaps enhance the stock HTML Buttons with a new attribute or something to control that feature.

@nairdo nairdo closed this as completed Jun 16, 2021
MrUpsideDown added a commit that referenced this issue Sep 27, 2021
MrUpsideDown added a commit that referenced this issue Oct 14, 2021
@crayzd92 crayzd92 added this to the v12 milestone Jan 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Confirmed It's clear what the subject of the issue is about, and what the resolution should be. Topic: Workflows Type: Enhancement Feature requests. x-Fixed in v12.7
Projects
None yet
Development

No branches or pull requests

4 participants