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

fix: include description match in rank #132

Merged
merged 7 commits into from
Nov 18, 2024

Conversation

barredterra
Copy link
Member

@barredterra barredterra commented Nov 18, 2024

  • voucher_number in description is now part of the queries and increases the rank
  • Only match the voucher number, not the entire voucher name
    ("9999" in description instead of "RE-9999" in description)
  • Order all query results by rank, so that top ranked results come first

@barredterra barredterra merged commit 623bbf7 into version-15 Nov 18, 2024
2 checks passed
@barredterra barredterra deleted the description-match-rank branch November 18, 2024 16:43
@barredterra
Copy link
Member Author

@Mergifyio backport version-14

Copy link

mergify bot commented Nov 18, 2024

backport version-14

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Nov 18, 2024
(cherry picked from commit 623bbf7)

# Conflicts:
#	banking/klarna_kosma_integration/doctype/bank_reconciliation_tool_beta/bank_reconciliation_tool_beta.py
barredterra added a commit that referenced this pull request Nov 18, 2024
Co-authored-by: Raffael Meyer <14891507+barredterra@users.noreply.github.com>
fix: include description match in rank (#132)
pull bot pushed a commit to ggraza/banking that referenced this pull request Nov 19, 2024
## [15.7.2](alyf-de/banking@v15.7.1...v15.7.2) (2024-11-18)

### Bug Fixes

* include description match in rank ([alyf-de#132](alyf-de#132)) ([623bbf7](alyf-de@623bbf7))
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.

1 participant