Skip to content

Query: Fixes FullText Policy API by making language optional #5172

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
May 7, 2025
Merged

Query: Fixes FullText Policy API by making language optional #5172

merged 5 commits into from
May 7, 2025

Conversation

adityasa
Copy link
Contributor

@adityasa adityasa commented May 6, 2025 •
edited
Loading

Query: Fixes FullText Policy API by making language optional

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • [] New feature (non-breaking change which adds functionality)
  • [] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [] This change requires a documentation update

Closing issues

To automatically close an issue: closes #IssueNumber

@adityasa adityasa marked this pull request as ready for review May 6, 2025 16:22
@adityasa adityasa added the auto-merge Enables automation to merge PRs label May 6, 2025
@microsoft-github-policy-service microsoft-github-policy-service bot enabled auto-merge (squash) May 6, 2025 22:56
@adityasa adityasa force-pushed the users/aditya/FTLanguageOptional branch from 4e55fdd to 30e0996 Compare May 7, 2025 04:17
Pilchie
Pilchie previously approved these changes May 7, 2025
View reviewed changes
kundadebdatta
kundadebdatta previously approved these changes May 7, 2025
View reviewed changes
Copy link
Member

@kundadebdatta kundadebdatta left a comment

Choose a reason for hiding this comment

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

LGTM Now.

@adityasa adityasa dismissed stale reviews from Pilchie and kundadebdatta via 9652ffc May 7, 2025 17:54
@adityasa adityasa force-pushed the users/aditya/FTLanguageOptional branch from 00ed3ca to 9652ffc Compare May 7, 2025 17:54
@adityasa adityasa requested a review from a team as a code owner May 7, 2025 17:54
@microsoft-github-policy-service microsoft-github-policy-service bot merged commit 9dd94ec into master May 7, 2025
26 checks passed
@microsoft-github-policy-service microsoft-github-policy-service bot deleted the users/aditya/FTLanguageOptional branch May 7, 2025 21:07
This was referenced Jul 28, 2025
Closed
Closed
This was referenced Jul 28, 2025
Closed
Open
Open
This was referenced Aug 5, 2025
Open
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Pilchie Pilchie Pilchie approved these changes

@kirankumarkolli kirankumarkolli kirankumarkolli approved these changes

@kundadebdatta kundadebdatta kundadebdatta approved these changes

@FabianMeiswinkel FabianMeiswinkel Awaiting requested review from FabianMeiswinkel FabianMeiswinkel is a code owner

@sboshra sboshra Awaiting requested review from sboshra sboshra is a code owner

@khdang khdang Awaiting requested review from khdang khdang is a code owner

@neildsh neildsh Awaiting requested review from neildsh neildsh is a code owner

@kirillg kirillg Awaiting requested review from kirillg kirillg is a code owner

Assignees
No one assigned
Labels
auto-merge Enables automation to merge PRs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@adityasa @Pilchie @kirankumarkolli @kundadebdatta