-
Notifications
You must be signed in to change notification settings - Fork 907
feat: Add Advanced UI Action Patterns #2416
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
feat: Add Advanced UI Action Patterns #2416
Conversation
- Add bulk operations manager for mass record updates - Add conditional action framework for dynamic UI behaviors - Add interactive form controller for enhanced user interactions - Add workflow integration handler for process automation - Include comprehensive documentation and usage examples These patterns demonstrate advanced ServiceNow UI Action techniques for: - Bulk data operations with progress tracking - Context-aware conditional actions - Dynamic form interactions and validations - Seamless workflow integrations
wiz0floyd
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.
Can you please justify why elements are being added directly to the DOM instead of using GlideModal to render them?
| `; | ||
|
|
||
| // Add to page (would typically be in a modal or dedicated area) | ||
| document.body.appendChild(progressContainer); |
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.
Dom manipulation is bad practice
| * - Memory-optimized record handling | ||
| * - Error handling and recovery | ||
| * | ||
| * @author ServiceNow Developer Community |
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.
What's this mean?
| if (g_user.hasRole('admin') || g_user.hasRole(tableName + '_admin')) { | ||
| operations.push({ | ||
| id: 'bulk_update', | ||
| name: 'Bulk Update Fields', |
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.
This is available oob for admins from list view.
|
👋 Unassigning @ravichandra1998g due to inactivity (> 60 min without comments/reviews). This PR remains open for other reviewers. |
|
Hello @ashvin2005 The submitted files contain hundreds of lines of code with multiple functions/patterns which makes them difficult to understand even with documentation. Please organize and submit smaller code snippets for specific use cases under dedicated folders (e.g., Catalog Client Script, UI actions, Business Rule, etc.) along with a clear README file explaining the use case and the purpose of each script. The current code is not ideal for code snippet submissions and make it harder to reuse/explore the code effectively. Closing this PR. |
These patterns demonstrate advanced ServiceNow UI Action techniques for:
PR Description:
replace this with your description
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