Skip to content

Prevent unnecessary notifications from being sent out #1516

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

Conversation

koratalas
Copy link
Contributor

Created a background script to prevent unnecessary notifications from being sent out. It helps in managing the volume of emails being sent so that we do not send the notifications even by mistake. This script is mostly used in dev or uat to avoid any notifications being sent from lower instances.

Created a background script to prevent unnecessary notifications from being sent out. It helps in managing the volume of emails being sent so that we do not send the notifications even by mistake. This script is mostly used in dev or uat to avoid any notifications being sent from lower instances.
readme.ms file has the instructions on how the script is working.
@Lacah Lacah self-assigned this Oct 28, 2024
@Lacah
Copy link
Contributor

Lacah commented Oct 28, 2024

Thanks for your contribution. Since you are making all the same updates to the GlideRecords, have you thought about using the updateMultiple() method instead :)

Updated the update method to updateMultiple method as per Lazlo's instructions.
@koratalas
Copy link
Contributor Author

Thanks Lazlo, changed the method to updateMultiple()

Copy link
Contributor

@Lacah Lacah left a comment

Choose a reason for hiding this comment

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

See in-line comment

@koratalas koratalas requested a review from Lacah October 29, 2024 16:16
@Lacah
Copy link
Contributor

Lacah commented Oct 29, 2024

Thanks!

@Lacah Lacah merged commit 185759c into ServiceNowDevProgram:main Oct 29, 2024
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