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

Loading screen visible when user taps "Sharing Location" #4022

Closed
mstidham opened this issue Aug 16, 2019 · 6 comments
Closed

Loading screen visible when user taps "Sharing Location" #4022

mstidham opened this issue Aug 16, 2019 · 6 comments
Assignees
Labels
Required for Release Required for the next app/play store release.

Comments

@mstidham
Copy link

Loading screen visible when user taps "Sharing Location"

QA TEMPLATE Input
Device Type: iPhone 7 Plus & Galaxy S10+
tr Version: 4.20.1
Environment Prod & Staging
Codepush:
Internet Connection Type: WiFi
Users affected? Miranda
User handle: @miranda
Bypass handle?:
Location access type? Always
Does this issue occur on other environments? Staging & Prod
Time issue occurred (UTC)

Observed Result:

If user taps on "Sharing Location" they briefly see the loading screen.

Expected Result:

User should only see this screen when opening the app

Steps to reproduce:

  1. Tap on "Sharing Location" from owners profile

Related Screenshots/Video Links:


Video:
https://app.box.com/s/qg8m7lv3xoja53z5dhxsn8n3oharo3ag

@mstidham
Copy link
Author

I'm still seeing this issue after the server fix. I also think the hit slop needs increased on the "Share Live Location" button.

@southerneer
Copy link
Contributor

Since this is routing related I think it makes sense for @aksonov to take this one.

@bengtan
Copy link
Contributor

bengtan commented Aug 20, 2019

Did some testing.

Happens on iOS Staging 4.20.1.

Happens on android Staging 4.20.1.

Happens on iOS Staging 4.20.0

Does not happen on iOS Staging 4.19.2.

@bengtan
Copy link
Contributor

bengtan commented Aug 20, 2019

If I run: 4.20.0 + revert #3986, the problem doesn't happen. Looks like #3986 introduced it.

@bengtan bengtan added the Required for Release Required for the next app/play store release. label Aug 21, 2019
aksonov added a commit that referenced this issue Aug 22, 2019
southerneer added a commit that referenced this issue Aug 22, 2019
@mstidham
Copy link
Author

mstidham commented Aug 22, 2019

Verified on Staging Version: 4.21.0

@mstidham
Copy link
Author

Verified on Prod Version: 4.21.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Required for Release Required for the next app/play store release.
Projects
None yet
Development

No branches or pull requests

4 participants