Skip to content

Conversation

@sandervandevelde
Copy link
Contributor

I compared the JSON notations with https://learn.microsoft.com/azure/event-grid/event-filtering?WT.mc_id=AZ-MVP-5002324#stringcontains .

  • It seems the backticks are unnecessary inside the quotes
  • some unnecessary spaces
  • use of double-quotes is not consistent

I compared the JSON notations with https://learn.microsoft.com/azure/event-grid/event-filtering?WT.mc_id=AZ-MVP-5002324#stringcontains .

* It seems the backticks are unnecessary inside the quotes
* some unnecessary spaces
* use of double-quotes is not consistent
@prmerger-automator
Copy link
Contributor

@sandervandevelde : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit f221886:

✅ Validation status: passed

File Status Preview URL Details
articles/event-grid/mqtt-routing-filtering.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@Jak-MS
Copy link
Contributor

Jak-MS commented Oct 24, 2023

@george-guirguis

  • Can you review this PR?
  • IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label Oct 24, 2023
@george-guirguis
Copy link
Contributor

#sign-off

@ShannonLeavitt
Copy link
Contributor

Thanks @sandervandevelde

@ShannonLeavitt ShannonLeavitt merged commit 20759ed into MicrosoftDocs:main Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants