Skip to content

Conversation

@poojithavalli22
Copy link
Contributor

PR Description:

Smart Form Validation for Catalog Items — a reusable Catalog Client Script that provides real-time field validation and configurable validation rules. The feature includes a single consolidated implementation with support for common validation types (email, phone, number, date), special validations (password strength, future date, working hours), custom validation types via variable naming, field dependency hooks, and a validation summary dialog. Full usage instructions and examples are included in the README.

Pull Request Checklist

Overview

  • Put an x inside of the square brackets to check each item.
  • I have read and understood the CONTRIBUTING.md guidelines
  • My pull request has a descriptive title that accurately reflects the changes and the description has been filled in above.
  • I've included only files relevant to the changes described in the PR title and description
  • I've created a new branch in my forked repository for this contribution

Code Quality

  • My code is relevant to ServiceNow developers
  • My code snippets expand meaningfully on official ServiceNow documentation (if applicable)
  • I've disclosed use of ES2021 features (if applicable)
  • [] I've tested my code snippets in a ServiceNow environment (where possible)

Repository Structure Compliance

  • I've placed my code snippet(s) in one of the required top-level categories:
    • Core ServiceNow APIs/
    • Server-Side Components/
    • Client-Side Components/
    • Modern Development/
    • Integration/
    • Specialized Areas/
  • I've used appropriate sub-categories within the top-level categories
  • Each code snippet has its own folder with a descriptive name

Documentation

  • I've included a README.md file for each code snippet
  • The README.md includes:
    • Description of the code snippet functionality
    • Usage instructions or examples
    • Any prerequisites or dependencies
    • (Optional) Screenshots or diagrams if helpful

Restrictions

  • My PR does not include XML exports of ServiceNow records
  • My PR does not contain sensitive information (passwords, API keys, tokens)
  • My PR does not include changes that fall outside the described scope

@rohi-v rohi-v self-assigned this Oct 19, 2025
@rohi-v
Copy link
Contributor

rohi-v commented Oct 19, 2025

Hi @poojithavalli22

Thank you for your contribution.

Please create separate js file for script include in the same folder.

@rohi-v
Copy link
Contributor

rohi-v commented Oct 19, 2025

Hi @poojithavalli22

Both the js files have the similar codes, Please provide the client script code in one js file and script include code in seperate js file.

I have also noticed issues in the code, Please ensure your code is working properly.

  1. Script Include call from client side should be done with GlideAjax.
  2. The Script Include should be client callable.
  3. g_form should not be used in server side.

Closing this for now. Once you make additional changes, feel free to re-open this Pull Request or create a new one.

@rohi-v rohi-v closed this Oct 19, 2025
@poojithavalli22 poojithavalli22 deleted the feature/smart-form-validation branch October 19, 2025 13:37
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