Skip to content

Conversation

@Suri123789
Copy link
Contributor

@Suri123789 Suri123789 commented Oct 28, 2025

PR Description:

I have created a client script in the new folder for validation of an interaction.

This Client Script ensures proper validation when resolving an Interaction record in ServiceNow.
It prevents a user from marking an Interaction as Closed Complete without proper justification.

also please consider only 2 files created on 28 October 2025 with names script.js and readme.md
Please ignore/delete "Code for Moveworks bot messages for Flow to include url links"(21b615f)

I am unable to delete that file. please help me to delete the same and accept this pull request

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

@ravichandra1998g ravichandra1998g self-assigned this Oct 28, 2025
@Suri123789
Copy link
Contributor Author

I have created a client script in the new folder for validation of an interaction.

This Client Script ensures proper validation when resolving an Interaction record in ServiceNow.
It prevents a user from marking an Interaction as Closed Complete without proper justification.

also please consider only 2 files created on 28 October 2025 with names script.js and readme.md
Please ignore/delete "Code for Moveworks bot messages for Flow to include url links"(21b615f)

I am unable to delete that file. please help me to delete the same and accept this pull request

@Suri123789 Suri123789 closed this Oct 28, 2025
@ravichandra1998g
Copy link
Contributor

Hello @Suri123789

Please remove the extra file as merging the PR merges 3 files you have added. You'll have an option to delete the file from PR, please check

@Suri123789
Copy link
Contributor Author

unable to delete the file. i have checked the same, could you please help me to delete the file

@Suri123789 Suri123789 reopened this Oct 28, 2025
@ravichandra1998g
Copy link
Contributor

Hello @Suri123789

I see you have deleted the 2 files now.

Can you please add the readme file.

@ravichandra1998g
Copy link
Contributor

Hello @Suri123789

Did you get a chance to check?

@Suri123789
Copy link
Contributor Author

Hi @ravichandra1998g

I have added the correct files now, please check and approve.

@ravichandra1998g
Copy link
Contributor

ravichandra1998g commented Oct 28, 2025

Hello @Suri123789 looks like you have created a new folder client scripts under Code snippets.

We already have client scripts under client side components under code snippets Repo.

Please DO NOT create new folders. use the relevant existing sub folders to place the file.

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