Skip to content

search issue with default radius#1315

Merged
kowsar89 merged 1 commit intoalphafrom
fix/radius-search
Dec 26, 2022
Merged

search issue with default radius#1315
kowsar89 merged 1 commit intoalphafrom
fix/radius-search

Conversation

@vairafiq
Copy link
Contributor

@vairafiq vairafiq commented Dec 26, 2022 •
edited
Loading

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Security fix
  • Improvement
  • New Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Text changes
  • Other... Please describe:

Description

How to reproduce the issue or how to test the changes

  1. Setup a default radius for search
  2. Hit search on Search Home
  3. Now you are on the search result page with all results, try searching listings using geolocation
  4. Reset the geo location and hit search, there should be all results again but not

Any linked issues

Fixes https://tasks.hubstaff.com/app/organizations/37274/projects/265387/tasks/5032458

Checklist

@vairafiq vairafiq requested a review from kowsar89 as a code owner December 26, 2022 04:18
@vairafiq vairafiq added this to the v7.5.0 milestone Dec 26, 2022
kowsar89
kowsar89 approved these changes Dec 26, 2022
View reviewed changes
Copy link
Contributor

@kowsar89 kowsar89 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG

@kowsar89 kowsar89 merged commit 2072ee6 into alpha Dec 26, 2022
@kowsar89 kowsar89 deleted the fix/radius-search branch December 26, 2022 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@kowsar89 kowsar89 kowsar89 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v7.5.0

Development

Successfully merging this pull request may close these issues.

2 participants

@vairafiq @kowsar89