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.

Sign up for GitHub

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

Jump to bottom

Tenant Home Page - Catalog search leads to an error #38

Open
6 tasks
Tracked by #33
jillpe opened this issue May 30, 2024 · 4 comments
Open
6 tasks
Tracked by #33

Tenant Home Page - Catalog search leads to an error #38

jillpe opened this issue May 30, 2024 · 4 comments
Assignees
@kirkkwang
Labels
bug something isn't working High Priority

Comments

@jillpe
Copy link

jillpe commented May 30, 2024 •
edited
Loading

Summary

Trying to do a catalog search from the tenant home page leads to a 'We're sorry something went wrong' page.

Screen.Recording.2024-05-29.at.7.53.27.PM.mov

Acceptance Criteria

  • Completing a catalog search from tenant home page directs the user to the search results page
  • I can see a search bar
  • I can customize the search results by selecting different filters (Sort by, number of records per page, layout)
  • I can click Start Over, removing all active filters
  • If there are multiple pages, I can see a pagination menu at the bottom of the page
  • I can limit my search by selecting from the metadata dropdown menus on the left
@jillpe jillpe mentioned this issue May 30, 2024
Open
19 tasks
@laritakr
Copy link
Contributor

A previously attempted fix apparently didn't resolve this.

#42

@ShanaLMoore ShanaLMoore added High Priority bug something isn't working labels Jun 3, 2024
@ShanaLMoore
Copy link
Contributor

Error:

Image

kirkkwang pushed a commit that referenced this issue Jun 21, 2024
🐛 Fix catalog searching
2ecc344 
The `#truncate_and_iconify_auto_link` method from
`Hyrax::OverrideHelperBehavior` was not being loaded.  I added an
include statement to the HykuKnapsack::ApplicationHelper which solved
the problem, though I thought this wasn't needed.

Ref:
  - #38
@kirkkwang kirkkwang mentioned this issue Jun 21, 2024
Merged
@kirkkwang kirkkwang self-assigned this Jun 21, 2024
@ShanaLMoore
Copy link
Contributor

May be related to #108

@ShanaLMoore ShanaLMoore moved this to SoftServ QA in PalsKnapsack Aug 5, 2024
@jillpe
Copy link
Author

jillpe commented Aug 12, 2024

SoftServ QA: pass ✅

Screen.Recording.2024-08-12.at.8.27.59.AM.mov

@jillpe jillpe moved this from SoftServ QA to Client QA in PalsKnapsack Aug 12, 2024
@jillpe jillpe mentioned this issue Aug 15, 2024
Open
1 task
@laritakr laritakr moved this from Deploy to Production to Client Verification in PalsKnapsack Feb 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kirkkwang kirkkwang

Labels
bug something isn't working High Priority
Projects
PalsKnapsack
Status: Client Verification
Milestone
No milestone
Development

No branches or pull requests
4 participants
@ShanaLMoore @laritakr @kirkkwang @jillpe