-
Notifications
You must be signed in to change notification settings - Fork 173
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: improve matching for tel type parameter #3653
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #3653 +/- ##
========================================
- Coverage 2.07% 2.07% -0.01%
========================================
Files 89 89
Lines 5348 5354 +6
Branches 1491 1492 +1
========================================
Hits 111 111
- Misses 5119 5125 +6
Partials 118 118
☔ View full report in Codecov by Sentry. |
The old implementation assigns the same score for HOME,VOICE and VOICE for a tel property with parameter voice. This pull requests adds an aditional point for the items with the same length. Signed-off-by: Daniel Kesselberg <[email protected]>
439dbad
to
503ae11
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and works. The additional score makes sense.
/backport to stable5.4 |
Fix #3651, Close #3652
The old implementation assigns the same score for HOME,VOICE and VOICE for a tel property with parameter voice.
This pull requests adds an aditional point for the items with the same length.