search

UAlbertaALTLab / morphodict

Plains Cree Intelligent Dictionary
https://itwewina.altlab.app/
Apache License 2.0
22 stars 11 forks source link

Fix/cr on search #1054

Closed nienna73 closed 2 years ago

nienna73 commented 2 years ago

What's in this PR:

Solves #1046 You no longer need to press Return to see search results with a dictionary source selected.

This entailed moving a piece of the logic to an earlier place in the search process.

codecov-commenter commented 2 years ago

Codecov Report

Attention: Patch coverage is 27.27273% with 8 lines in your changes missing coverage. Please review.

Project coverage is 78.26%. Comparing base (50ccd84) to head (44ec02e). Report is 562 commits behind head on main.

Files Patch % Lines
src/CreeDictionary/API/search/presentation.py 27.27% 7 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1054 +/- ## ========================================== + Coverage 78.23% 78.26% +0.02% ========================================== Files 151 151 Lines 5394 5391 -3 Branches 865 862 -3 ========================================== - Hits 4220 4219 -1 + Misses 1043 1041 -2 Partials 131 131 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.