Skip to content
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

Update NRT_AuthenticationMethodsChangedforVIPUsers.yaml #11736

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

AndreasRogge
Copy link

Added square brackets for watchlist query, because column names with space cannot be referenced through simple ". You need to write the column in square brackets.

Required items, please complete

Change(s):

  • Updated syntax for NRT_AuthenticationMethodsChangedforVIPUsers.yaml

Reason for Change(s):

  • Error that you cannot reference the watchlist in question

Version Updated:

  • Yes

Testing Completed:

  • Yes

Checked that the validations are passing and have addressed any issues that are present:

  • No

Added square brackets to watchlist query because column names with spaces cannot be referenced by just using ".
@AndreasRogge AndreasRogge requested review from a team as code owners January 29, 2025 09:58
@v-atulyadav v-atulyadav added Solution Solution specialty review needed Analytic Rules labels Jan 29, 2025
@rahul0216
Copy link
Contributor

@AndreasRogge Please have a look at failing KQL validation and resolve it.

@AndreasRogge
Copy link
Author

@AndreasRogge Please have a look at failing KQL validation and resolve it.

Hi @rahul0216, the error seems to be coming from the non existing watchlist. So it is nothing that I can change because the watchlist does not exist for the test.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Analytic Rules Solution Solution specialty review needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants