Skip to content

Conversation

@SaiSankar1309
Copy link
Contributor

PR Description:

Business Service Fetch When CI Changes.
The earlier PR #1739 was closed before final changes were committed.
This version includes:

  • Corrected client script to handle the response properly
  • Cleaned up Script Include logic
  • Comments added for clarity
    CSDM alignment flagged as a future enhancement.

Pull Request Checklist

Overview

  • I have read and understood the CONTRIBUTING.md guidelines
  • My pull request has a descriptive title that accurately reflects the changes
  • 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 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

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

This code snippet fetches the business service related to the CI , when the CI is changed in the change_request form.
@rohi-v rohi-v self-assigned this Oct 3, 2025
@SaiSankar1309
Copy link
Contributor Author

For now, keeping the scope focused on direct CI → Business Service relationships for clarity and testability.

Copy link
Contributor

@rohi-v rohi-v left a comment

Choose a reason for hiding this comment

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

Thank You for your Contribution

Please create seperate js files( 1 for Script Include and 1 for Client script)

Once changes are done,I will review and update it.

Copy link
Contributor

@rohi-v rohi-v left a comment

Choose a reason for hiding this comment

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

Thank You for your contribution

Please create seperate js files for SI and Client script

Once chages are done, I will review and approve it.

created a separate file for onChange client script
@SaiSankar1309
Copy link
Contributor Author

Hello Rohi,
Thank you for your feedback. Client script has been placed in a separate .js file.

@rohi-v
Copy link
Contributor

rohi-v commented Oct 3, 2025

Looks Good

Thank you for your contribution

@rohi-v rohi-v closed this Oct 3, 2025
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