project-lux / lux-frontend

Web front end of LUX
Apache License 2.0
3 stars 0 forks source link

Investigate Site Improve Bugs related to Landing Page. #316

Open roamye opened 3 months ago

roamye commented 3 months ago

Problem Description: Issue 222 fixed the majority of the issues listed within the ticket. However, the remaining issue that continues to show "Unknown Page Name". This is caused by poor state management in the frontend.

Expected Behavior/Solution: The frontend will be updated to allow SiteImprove to capture all page names. "Unknown page name" should no longer appear in SiteImprove unless an error occurs.

Requirements:

Needed for promotion: If an item on the list is not needed, it should be crossed off but not removed.

UAT/LUX Examples:

Dependencies/Blocks: N/A

Related Github Issues:

Related links: N/A

Wireframe/Mockup: N/A

kamerynB commented 3 months ago

@roamye For the first issue, selecting the "Apply" button, I see that it works in DEV. I've included a screenshot below of both the event and subsequent events. Below are the steps I took to get there.

  1. Go to landing page
  2. Search "andy warhol"
  3. Select the Creation Date facet
  4. Select a range
  5. Select "Apply" (this event is captured in the first screenshot
  6. Select the first returned results (this event is shown in the second screenshot)

Screenshot 1: Screenshot 2024-08-02 at 11 47 35 AM

Screenshot 2: Screenshot 2024-08-02 at 12 02 31 PM

roamye commented 2 months ago

@kamerynB - you are correct! It works now so I updated the issue since the date facet is no longer an issue.

kamerynB commented 2 months ago

@roamye Thank you! I will continue looking into the landing page issue though.

roamye commented 1 month ago

From discussion with Kam 10/2/24:

Notes: Landing pages to....

As noted above, going directly to a link (non home page) and then selecting any other link will result in an "unknown page" ex: https://lux-front-dev.collections.yale.edu/view/person/85ab18e6-b208-4658-8ea4-abaca7a60812

roamye commented 4 weeks ago

From ITS Team Meeting 10/4/24: The solution above is not sufficient as we also need to get the correct label for starting on a non home page link. Next steps:

roamye commented 2 weeks ago
  1. Landing page to Submit Simple Search | works well. recognizes it came from the lux landing page.
  2. Any link to Landing Page | works well. recognizes where it came from.
  3. Entry page (record) to open access | works well. example
  4. internal link to entity link | works well .example
  5. AS to any link | works well.

    I tested the scenarios above + similar tests with different links and it all looks good to me. Is there any specific scenario I should test? @kamerynB

roamye commented 1 week ago

Approved by UAT