feat: FORMS-25252 add server-side validation (SSV)#1902
Conversation
Accessibility Violations Found
|
10d09e8 to
3d10aa4
Compare
Accessibility Violations Found
|
Accessibility Violations Found
|
2 similar comments
Accessibility Violations Found
|
Accessibility Violations Found
|
| submitProps.computeIfAbsent(SS_SPREADSHEET, k -> new LinkedHashMap<String, Object>()); | ||
| ((Map<String, Object>) submitProps.get(SS_SPREADSHEET)).put(entry.getKey(), entry.getValue()); | ||
| } | ||
| // SSV properties (enableServerSideValidation, ssvCloudServicePath) are intentionally |
There was a problem hiding this comment.
nitpick: comment not required here
| granite:class="cmp-adaptiveform-container__enablessv" | ||
| name="./enableServerSideValidation" | ||
| text="Enable Server-Side Validation" | ||
| value="true" |
There was a problem hiding this comment.
I think default value should be false
There was a problem hiding this comment.
Yes made it false.
Accessibility Violations Found
|
1 similar comment
Accessibility Violations Found
|
| public static final String PN_SUBMIT_EMAIL_CC = "cc"; | ||
| public static final String PN_SUBMIT_EMAIL_BCC = "bcc"; | ||
| public static final String PN_SUBMIT_SPREADSHEETURL = "spreadsheetUrl"; | ||
| public static final String PN_ENABLE_SERVER_SIDE_VALIDATION = "enableServerSideValidation"; |
There was a problem hiding this comment.
all OOTB properties should be prefixed with fd:
There was a problem hiding this comment.
These properties should not be part of the model.json output since they are server side, there can be a use-case for EDS submit, that can be handled later
There was a problem hiding this comment.
These are not part of model.json; these are just for internal code use.
| </cui> | ||
| </uiSettings> | ||
| </thankYouMessage> | ||
| <enableServerSideValidation |
There was a problem hiding this comment.
This should be hidden and only enabled if someone selects enable server side validation
There was a problem hiding this comment.
Discussed with Anurag and Sudhanshu that, in order to improve the archaic behaviour, we should keep only one dropdown with a default value of None.
| jcr:primaryType="nt:unstructured" | ||
| sling:resourceType="granite/ui/components/coral/foundation/form/checkbox" | ||
| fieldLabel="Server-Side Validation" | ||
| fieldDescription="Call an Adobe I/O validator endpoint before the submit action runs." |
There was a problem hiding this comment.
We need to have help article for users to understand how to create one
There was a problem hiding this comment.
Yes I will be creating a Doc for the customer.
| let defaultSubmissionError = FormView.LanguageUtils.getTranslatedString(self.getLang(), "InternalFormSubmissionError"); | ||
| const globals = { | ||
| form: self.getModel().getRuleNode(), | ||
| formModel: self.getModel(), |
There was a problem hiding this comment.
Its not required. Removed it.
| <enableServerSideValidation | ||
| jcr:primaryType="nt:unstructured" | ||
| sling:resourceType="granite/ui/components/coral/foundation/form/checkbox" | ||
| fieldLabel="Server-Side Validation" |
There was a problem hiding this comment.
Can we create a github clone so that users can refer that as sample to deploy Adobe IO action ? I want to make sure that they dont spend a lot of time doing this
There was a problem hiding this comment.
Yes, I will be creating one.
7b1c652 to
665f05d
Compare
Accessibility Violations Found
|
1 similar comment
Accessibility Violations Found
|
Accessibility Violations Found
|
1 similar comment
Accessibility Violations Found
|
Accessibility Violations Found
|
1 similar comment
Accessibility Violations Found
|
b0589ab to
e397c06
Compare
Accessibility Violations Found
|
3 similar comments
Accessibility Violations Found
|
Accessibility Violations Found
|
Accessibility Violations Found
|
Accessibility Violations Found
|
Accessibility Violations Found
|
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
Accessibility Violations Found
|
1 similar comment
Accessibility Violations Found
|
Accessibility Violations Found
|
Accessibility Violations Found
|
0068aba to
a69124c
Compare
Accessibility Violations Found
|
1 similar comment
Accessibility Violations Found
|
| } | ||
|
|
||
| private List<Resource> getCloudConfigsByGroup(SlingHttpServletRequest request, ResourceResolver resourceResolver) { | ||
| List<Resource> resources = new ArrayList<>(); |
There was a problem hiding this comment.
This is incorrect, we have helpers in cq-guides (addon) for such use-cases, you should re-use that
rismehta
left a comment
There was a problem hiding this comment.
Approving. but there are few unresolved comments, address them later
Accessibility Violations Found
|
Accessibility Violations Found
|
60d88c2 to
45900ec
Compare
45900ec to
b86b3ea
Compare
Accessibility Violations Found
|
3 similar comments
Accessibility Violations Found
|
Accessibility Violations Found
|
Accessibility Violations Found
|
* FORMS-23760 : Added Table component support in Core Components (#1879) * added basic config files for the table component * added .content.xml for the table component * added edit toolbar for the table component _cq_editConfig.xml * added table.html * added basic css and js for the table authoring currently refering the dependencies which are declared in the content.xml * added site clientlibs for the table component * added basic config files for the tableheader * added basic config files for the tablerow * added site clientlibs for tableheader * added clientlibs folder in the tablerow * added the table component in examples folder along with cq_template * added runtime dependencies in the runtime all for the table component * replaced the tr elements from the div elements to fix the selecting issue * added action configs in the edit toolbar for the table component * added action configs for the table row and the tableroweditorhook.js for handling this * removed the add row option from the tableHeaderRow * updated the header cells from the textinput to text type in the templates * added the addCol and deleteCol actions and also there handlers * removed cell numbers from dynamic addition of the table cols and rows * added the feature of repeatable panels to the table row at runtime * added the replace feature for the tablecell to any component which is present * removed the delete options from the table cells in the table row * in this commit table tags have gone -> repeatable panels not working correctly here * added files to fix the repeatative panels with the table tags * fixed the repeatable rows with indexing * changed the authoring from divs to tabletags sacirificing the touch ui for now * fixed the touch ui with the table tags * fixed the tablecell css having controls * removed the tableeditor.css file * added replace feature to the components of the table Header also * fixed the refresh issue by adding authoring dialogs for table row and table header * added cypress tests and collaterals for testing * added sling folder for collaterals * fixed the test cases * fixed the content.xml * fixed the failing testcases * updated the year to 2026 * fixed the short and long description for the table component * removed unsupported authoring from the table component * refactored the table.html * updated README for the table Component * Resolved Conflicts * added sorting feature for the table component * enabled sorting button during authoring too * added feature to disable/enable sorting on a particular column of the table * added authoring options for sorting and column width * added dor support for table component * added dor testing * added the feature of merge and split cell during runtime by adding colspan in publish html * updated README for table, tableheader, tablerow * added the feature for merge row cells and commented out the header cells merge feature * added testing for second phase features * fixed review comments 1 * fixed review comments 2 * fixed cypress tests * fixed coverage * fixed the sorting behaviour to improve the archaic behavior * fixed cypress testing mergesplit feature * fixed cypress testing * moved .css changes from core component to theme canvas --------- Co-authored-by: Armaan Gupta <armaang@Armaans-MacBook-Pro.local> * FORMS-24585 AEP submission changes (#1911) * Fix code formatting for FormContainerImpl.java (#1914) * FORMS-24585 Adding sandboxname in config (#1917) Co-authored-by: shraddhasingh683 <shraddhasingh683@gmail.com> * Double MB fix in validation alert pop up for file size (#1918) * file input upload error fix * double MB java change reverted --------- Co-authored-by: Rajat Khurana <rajatkhurana@adobe.com> * feat: FORMS-25252 add server-side validation (SSV) (#1902) * feat(af-core): FORMS-25252 add server-side validation (SSV) for form container * fix(ssv): gate SSV dialog behind FT_FORMS-25252 feature toggle * copy writing fixed for ssv config --------- Co-authored-by: Rajat Khurana <rajatkhurana@adobe.com> --------- Co-authored-by: Armaan Gupta <armaang@Armaans-MacBook-Pro.local> Co-authored-by: shraddhasingh683 <shraddhasingh683@gmail.com> Co-authored-by: anshikag-adobe <anshikag@adobe.com> Co-authored-by: Rajat Khurana <30727855+rajatofficial@users.noreply.github.com> Co-authored-by: Rajat Khurana <rajatkhurana@adobe.com>
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: