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

Crash when Trip with GPS coordinates is made a Favourite Trip #624

Open
bkqtnte10 opened this issue Jul 21, 2019 · 3 comments
Open

Crash when Trip with GPS coordinates is made a Favourite Trip #624

bkqtnte10 opened this issue Jul 21, 2019 · 3 comments
Labels
bug 🐞 A functional defect or unexpected behavior. needs info ℹ️ More information needed,
Milestone

Comments

@bkqtnte10
Copy link

Tapping the Star-Icon on a Trip with GPS coordinates crashes the app

To Reproduce

  1. select a point on the map (coordinates) and use it as "from here"
  2. make sure the coordinates don't get mapped to an address (for example by setting the phone to airplane mode)
  3. Choose any other location as destination
  4. exit airplane mode
  5. search for connections (tap "try again")
  6. hit the star-icon
  7. See error

Expected behavior
Either

  • an error message, that coordinates cannot be set as favorite

or

  • successfully accepting the new favorite without an error

Screenshots
Screenshot_20190721-133653_Transportr

Versions:

  • Transportr Version: 2.0.6
  • Device: Xiaomi Mi 6
  • Android Version: LineageOS 8.1
@grote grote added the bug 🐞 A functional defect or unexpected behavior. label Jul 21, 2019
@grote grote added this to the 2.1 milestone Jul 21, 2019
@ialokim
Copy link
Collaborator

ialokim commented Mar 23, 2020

Either

  • an error message, that coordinates cannot be set as favorite

or

  • successfully accepting the new favorite without an error

@grote What would be the preferred behavior here?

@grote
Copy link
Owner

grote commented Mar 23, 2020

I think the expected behavior is to not offer the option to make a trip a favorite that isn't supported. However, there's nothing wrong supporting it. The idea was to use the geocoder to turn the coords into real addresses, but not all providers support the way we currently use: #517
In this case, the geocoding either failed or wasn't fast enough I guess.

@ialokim ialokim modified the milestones: 2.1, 2.2 Aug 4, 2020
@Altonss
Copy link
Collaborator

Altonss commented Dec 5, 2023

In which networks is it possible to use such GPS coordinates as input? Didn't manage to make a trip...

@Altonss Altonss added the needs info ℹ️ More information needed, label Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐞 A functional defect or unexpected behavior. needs info ℹ️ More information needed,
Projects
None yet
Development

No branches or pull requests

4 participants