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

Bug: Distance/date not working properly #20

Closed
5 tasks done
JuditKaramazov opened this issue Jun 30, 2024 · 0 comments · Fixed by #21
Closed
5 tasks done

Bug: Distance/date not working properly #20

JuditKaramazov opened this issue Jun 30, 2024 · 0 comments · Fixed by #21
Assignees
Labels
bug Something isn't working

Comments

@JuditKaramazov
Copy link
Owner

JuditKaramazov commented Jun 30, 2024

Describe the bug

Due to a recently added Barcelona pin, distance/date calculations are not as precise as they should - not even as they were.
Considering this, we will:

  • Investigate the specific pin to see if there’s wrong or missing information triggering this particular issue.
  • Ensure the rest of the application is working as intended.
  • Revise “sort by distance/date” behavior.

Steps to reproduce

  1. Go to the menu.
  2. Sort the pins by date or distance. A “Barcelona” one will usually appear first - and it shouldn’t, as the calculation is not correct.
  3. Close the menu.
  4. Go to the menu again.
  5. Sometimes, the order is correct, but sorting the pins may trigger this error again.

Do you have any screenshots?

Affected pin

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs
@JuditKaramazov JuditKaramazov added the bug Something isn't working label Jun 30, 2024
@JuditKaramazov JuditKaramazov self-assigned this Jun 30, 2024
@JuditKaramazov JuditKaramazov linked a pull request Jul 2, 2024 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant