Staging → Production
This merge request would bring the following MRs into prod:
-
home-fuzzy-search!131 (merged) -
145-answer-buttons-overlap!130 (merged) -
166-hide-exam-status!129 (merged) -
full-text-search!103 (merged) -
156-display-creation-time!123 (merged) -
increase-notification-interval!122 (merged) -
170-add-opacity-to-text-layer!124 (merged) -
157-prevent-empty-comments-and-answers!125 (merged) -
Add-UI-for-markexamchecked!116 (merged) -
remove-tutorial!117 (merged) -
172-add-option-to-view-source-code-of-an-answer!121 (merged)
!103 (merged) includes a migration that can take a while to process which requires disabling the liveness probe for the pod during migration.
Merge request reports
Activity
The discussion we had yesterday concerning the speed and accuracy of fuzzy search didn't fully resolve yet (imo). I love the feature as the first few (intended) results seem really accurate especially when considering typos but the others are rather bad. Would it be an option to change the score threshold?
Then there was the idea to only update the search when there hasn't been any action in a certain time range which should improve speed. Any thoughts on that?
- Resolved by Jason Zibung
(Disclaimer: see #145 (closed) I tested using chromium based responsive viewer and there is some weird stuff going)
Didn't we say in #145 (closed) that we should support down to 320px viewport width?
thus the MR !130 (merged) seems incomplete to me.
Edited by Jason Zibung - Last reply by Jason Zibung
- Resolved by Jason Zibung
Related to !103 (merged)
Again something that only seems to be broken in Chrome and Chromium based browsers (checked on different devices and browsers):
Top right result doesn't seem to be in a card even tho it has the appropriate class names.
Edited by Jason Zibung - Last reply by Jason Zibung
mentioned in merge request !133 (merged)
-
This comment serves the purpose of keeping track which MRs I checked and am content with:
-
!131 (merged)!135 (closed)!137 (merged) - !130 (merged) (it is okay how it is it's just not finished but this should not block this MR)
- !129 (merged)
- !103 (merged)
- !123 (merged)
- !122 (merged)
- !124 (merged)
- !125 (merged)
- !116 (merged)
- !117 (merged)
- !121 (merged)
Edited by Jason Zibung -
mentioned in issue #182 (closed)
added 15 commits
- a50b2a3a - Use custom fuzzy search
- fb515224 - Add some comments
- a46d5e8d - Explain scoring
- fd0dd97c - Rename scoring function
- 6412d0f0 - Add jsdoc
- 8d78118e - Fix formatting
- 5a5cb6a6 - Fix category card
- c22f73d2 - Improve search-utils variable naming
- 4a2137bc - Cache dp tables
- d9822d69 - Correct comment
- ee33f683 - Fix arrayStride / rows calculation
- 64cffbb3 - More comments
- e0f191c0 - Transpose array to have constant array stride
- 7222926e - Allow typos based on filter length
- 87e97038 - Merge branch 'custom-fuzzy' into 'staging'
Toggle commit listadded 7 commits
Toggle commit listmentioned in commit 7639ef73
