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 - "Not found page" not shown for Search page with invalid query parameters #56820

Closed
2 of 8 tasks
IuliiaHerets opened this issue Feb 13, 2025 · 3 comments
Closed
2 of 8 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2

Comments

@IuliiaHerets
Copy link

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Version Number: v9.0.98-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail: #49539
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team
Device used: Mac 14.5/ Chrome
App Component: Search

Action Performed:

  1. Open App
  2. Open the search page with invalid query parameters (e.g /search?q=from%3a)

Expected Result:

The not found page is shown

Actual Result:

The not found page is not shown

Workaround:

Unknown

Platforms:

  • Android: Standalone
  • Android: HybridApp
  • Android: mWeb Chrome
  • iOS: Standalone
  • iOS: HybridApp
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Bug6741496_1739422952794.Screen_Recording_2025-02-13_at_7.48.06_in_the_morning.mp4

View all open jobs on GitHub

@IuliiaHerets IuliiaHerets added Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 labels Feb 13, 2025
Copy link

melvin-bot bot commented Feb 13, 2025

Triggered auto assignment to @greg-schroeder (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@Kicu
Copy link
Member

Kicu commented Feb 14, 2025

IMO ?q=from%3a is not an invalid query parameters. This is simply empty from: clause. Yes, the search will return nothing in that case, but it does not break it.

For any query that technically is correct (no weird characters, no errors) but makes no logical sense (like search for something empty) we have decided to simply display empty results view.
Not sure if we want to change it to full "Not found" page 🤔

CC @luacmartins

@luacmartins
Copy link
Contributor

Yea, I don't think this is a bug. We're working on a separate issue to show an error message if the user does indeed submit an invalid query and we get a 5xx error back from the server. We can close this issue!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2
Projects
None yet
Development

No branches or pull requests

4 participants