-
-
Notifications
You must be signed in to change notification settings - Fork 73
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
Not all files to download are visible #902
Comments
@kelson42 as of my initial findings it seems that we use custom DB query for these results, which is currently searching (in case of this "New" section) only: kiwix-apple/Views/Library/ZimFilesNew.swift Lines 97 to 109 in 7d4b5b7
It would be great to know how exactly the website search works, and we can adjust the query to match that. |
My other, slightly related concern about how the search works is that it's really hard to find a ZIM that is not "new". Currently the user can search only within the selected category (eg. "Other"), but that is a very strict rule, I think. We assume that the user knows which category that ZIM belongs to... |
Reopening as I believe we should do more than this. @BPerlakiH Thank you for your englighting comments, I will come back to you |
@BPerlakiH I agree. Please open an issue just for that and put it in next milestone. |
@BPerlakiH To complete this PR, can you change 1 month in 3 months because 1 month is really too short. |
Increased the lifespan of "NEW" to be 3 months here: #910 |
new issue created here: #911 |
Did the refresh solve the issue? |
No |
Can you do a screenshot of the "New" section when the search query is empty, and also one screenshot of "Settings > Languages" ? |
Both of those results are in "English" only, so if you do not have English selected, there won't be any result for that query. |
Works now with latest testflight of this morning... somI guess the fix was still not included. Closing. |
If I search for "bug" online, I get:
If I search for "bug" in library, I get:
I should not get different resulta IMHO, at least not for this search!
The text was updated successfully, but these errors were encountered: