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

Search in the UV not working #175

Closed
KatharineV opened this issue Apr 12, 2024 · 6 comments
Closed

Search in the UV not working #175

KatharineV opened this issue Apr 12, 2024 · 6 comments
Labels
bug something isn't working Knapsack Upgrade

Comments

@KatharineV
Copy link
Collaborator

After Knapsack, the search box in the UV is not working. The example document below contains the word "work" multiple times, so I assume that Tesseract caught at least one of those instances. The word "work" should be searchable. When I enter the word in the search box and hit enter, I get a spinning circle that never resolves. The word never highlights and I don't get an error message or anything.

https://adl.b2.adventistdigitallibrary.org/concern/generic_works/wdf_0010_005_the_nashville_sanitarium_ellen_g_white_extracts_from_lette?locale=en
Image

@KatharineV KatharineV added bug something isn't working Knapsack Upgrade labels Apr 12, 2024
@ShanaLMoore
Copy link

ShanaLMoore commented Apr 13, 2024

This wasn't working due to mixed content error seen in the console. Once I marked the tenant as ssl_configured: true, the search works again - No code changes were needed.

Image

Image

@KatharineV
Please give it a go and let us know if there's still a problem.

ref:

@KatharineV
Copy link
Collaborator Author

I tested on ADL prod, and the UV search worked. Yay!

But the UV search is still not working in staging.
https://testing.s2.adventistdigitallibrary.org/concern/published_works/019887_an_explanation_of_the_new_folding_prophetic_chart
It just spins and never finds the term, although I can see the word on the page.

Image

@ShanaLMoore
Copy link

@KatharineV This is resolved by setting the ssl_configured to true for the tenant

ref:

Image

Image

@KatharineV
Copy link
Collaborator Author

Thanks, @ShanaLMoore . Is the Ssl feature one that I need to flip on for all our tenants? If so, can you tell me which part of the dashboard is showing in that screenshot above? I can go through all the tenants and make sure it's on.

@ShanaLMoore
Copy link

ShanaLMoore commented Apr 18, 2024

We were just talking about if we should consider creating a ticket to turn ssl_configured on by default for everyone since it definitely seems to be the way you'd need it.

You go to admin dashboard. In the left column click Settings > Account. Scroll all the way down and check the checkbox for ssl_configured. Click Save Changes.

demo: https://share.zight.com/RBuAKXd2

Image

@KatharineV
Copy link
Collaborator Author

Thanks! I've set Ssl configured to True for all the (in-use) tenants in B2 and S2.

@kirkkwang kirkkwang transferred this issue from notch8/adventist-dl May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something isn't working Knapsack Upgrade
Projects
Status: Done
Status: Done
Development

No branches or pull requests

3 participants