Fix: Update Q3 notification query to use Admin assignment in test 21820#1084
Merged
SagarSathe merged 3 commits intodevfrom Apr 22, 2026
Merged
Fix: Update Q3 notification query to use Admin assignment in test 21820#1084SagarSathe merged 3 commits intodevfrom
SagarSathe merged 3 commits intodevfrom
Conversation
praneeth-0000
added
ready for review
alexandair
requested changes
Apr 20, 2026
Collaborator
alexandair
left a comment
alexandair
left a comment
There was a problem hiding this comment.
@praneeth-0000 Please, address my feedback.
SagarSathe
approved these changes
Apr 22, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updates the Query 3 in test 21820 to use Notification_Admin_EndUser_Assignment in place of Notification_Requestor_EndUser_Assignment and changes the fail scenario logic, as mentioned in issue #1039.