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

Add EVENT result_mapping which doubles titles for configured SPs DO NOT MERGE #1119

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

sid-swirl
Copy link
Contributor

Branching Reminders

X For core code changes, create a branch off develop

  • For product documentation changes, create a branch off main
  • Always name your branch in a way that clearly describes your proposed changes

Description

Apply to Outlook Calendar e.g.
"...,EVENT, FILESYSTEM..."

Related Issue(s)

Testing and Validation

Tested extensively by M. West

Type of Change

  • Bug fix or other non-breaking change that addresses an issue
  • New Feature / Enhancement (non-breaking change that add or improves functionality)
  • New Feature (breaking change that is not backwards compatible and/or alters current functionality)
  • Documentation (change to product documentation or README.md only)

We recommend applying to Outlook Calendar
@sid-swirl sid-swirl requested a review from dnicodemus January 26, 2024 04:19
Copy link

@kiruthikpurpose kiruthikpurpose left a comment

Choose a reason for hiding this comment

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

Great job on implementing the EVENT result mapping. The addition of doubling titles for configured SPs enhances the relevance of search results. Just be cautious with the boosting logic to ensure it works as intended across all scenarios.

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