Skip to content

Conversation

@anurampalli
Copy link
Contributor

... snippet with README and screenshots

PR Description:

replace this with your description

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

...  snippet with README and screenshots
@am-shakeel am-shakeel self-assigned this Oct 10, 2025
Copy link
Contributor

@am-shakeel am-shakeel left a comment

Choose a reason for hiding this comment

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

Hi @anurampalli, I see that you have added the OnBefore Transform Map Script screenshot. Could you please add the script file for the same?

@anurampalli
Copy link
Contributor Author

hi Shakeel, I think if I do that, I will have to do another PR and a README/ Can I just leave it in this folder as a text file? or put it in the README?

@am-shakeel
Copy link
Contributor

And also, the Readme-title is not needed. It is better to have one Readme file only. Please try to use the Readme-title file for OnBefore. Otherwise, I will close this and create a new PR with all the details.

@anurampalli
Copy link
Contributor Author

Okay, thank you. Let me try. Or I can do another PR. Thanks !

…n Transform Map/README-title.md

Deleting unnecessary file
@am-shakeel
Copy link
Contributor

am-shakeel commented Oct 10, 2025

Hi @anurampalli, are you able to make the changes, or are you creating a new PR?

Updated with usage example (onBefore script) and screenshots
@anurampalli
Copy link
Contributor Author

I did the changes. Thank you for your patience :-)

@am-shakeel
Copy link
Contributor

The OnBefore script should be in JS format, just like the OnStart Transform Map script.

@anurampalli
Copy link
Contributor Author

I am not sure I can upload another file to a committed branch. I don't see that option in the github desktop. I will check one more time. Please give me a few minutes.

@anurampalli
Copy link
Contributor Author

Done, but I read somewhere that a fresh JS needs a fresh PR. Please can you confirm.

Copy link
Contributor

@am-shakeel am-shakeel left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for the contribution.

@am-shakeel am-shakeel merged commit d2f5d82 into ServiceNowDevProgram:main Oct 10, 2025
1 check passed
@anurampalli
Copy link
Contributor Author

Looks good, thanks for the contribution.

Great! Thank you so much!!!

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