Skip to content

Conversation

@SumithThota
Copy link
Contributor

@SumithThota SumithThota commented Oct 21, 2025

PR Description:

Fixes Issue #2367
Provided correct readme to the files 1, 2 and 3

Also do check this PR #2357 since it closed due to the present Issue

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

@SumithThota
Copy link
Contributor Author

Accidentally removed a space after Background scripts
Original Path: Server-Side Components/Background Scripts/ Bulk Change of Incident Priority Based on Category/Bulk Change of Incident Priority Based on Category.js
New Path: Server-Side Components/Background Scripts/Bulk Change of Incident Priority Based on Category/Bulk Change of Incident Priority Based on Category.js
The only difference is the removal of a space after the /Background Scripts/ portion of the path.
I belive it may be due to the file paths

@ravichandra1998g ravichandra1998g self-assigned this Oct 21, 2025
@ravichandra1998g
Copy link
Contributor

Hello @SumithThota

Can you please confirm this doesn't create a dummy path under the background scripts folder?

@SumithThota
Copy link
Contributor Author

Hi @ravichandra1998g I have checked whether there exists any dummy paths in the branch that I have created for this PR
but there is no such paths
You can have a look over here
image

Copy link
Contributor

@ravichandra1998g ravichandra1998g 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!

@ravichandra1998g ravichandra1998g merged commit 3aa97b0 into ServiceNowDevProgram:main Oct 21, 2025
2 checks passed
@SumithThota SumithThota mentioned this pull request Oct 21, 2025
17 tasks
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