-
Notifications
You must be signed in to change notification settings - Fork 906
simplified #2531
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
simplified #2531
Conversation
Atul-LNG
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add the Readme file as well.
|
This was just a simplification of a script. It already has a README |
Atul-LNG
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
README must be a separate file @ChaseMillers as per guidelines.
|
There IS a separate REAME file to this folder. You don't want two of them. This is based off a script that was already made. It was made by me. |
|
@Atul-LNG |
I'll be sure to include the PR references next time. Thanks! |
PR Description:
Previous script, now refactored:
Manual parent table traversal and sys_dictionary queries are replaced with ServiceNow’s built-in [GlideRecord.getElements()] for listing all fields.
Field existence checks now use [GlideRecord.isValidField(fieldName)].
Table ancestors are printed using [SNC.TableEditor.getTableAncestors()] if available.
test:


Pull Request Checklist
Overview
Code Quality
Repository Structure Compliance
Core ServiceNow APIs/Server-Side Components/Client-Side Components/Modern Development/Integration/Specialized Areas/Documentation
Restrictions