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

reorganize settings again - degrade autosync #1455

Closed
wants to merge 1 commit into from

Conversation

matkoniecz
Copy link
Member

@matkoniecz matkoniecz commented Jun 30, 2019

autosync is not an important option, and enabling it may be problematic - for example one of the complaints in English reviews on Google Play is from person confused by interface that used to be default and now appears only with autosync disabled

autosync also requires people to manually sync what is increasing chance of unsynced data, or data synced very late

In addition advanced setting were moved around - moving to top ones that are more likely to be usefully used


My work on this pull request and UX testing was sponsored by a NGI Zero Discovery grant

@matkoniecz
Copy link
Member Author

This PR was triggered by reading through reviews in English and Polish on Google Play.

For all reading this - Is there some review of StreetComplete on Google Play in your language (that is not Polish nor English) that directly or indirectly has some legitimate complaint or useful suggestion? Without any issue covering this? In that case it may be useful to create issue to let know people about it, assuming that it is possible to create an useful one.

autosync is not an important option, and enabling it may be problematic - for example one of the complaints in English reviews on Google Play is from person confused by interface that used to be default and now appears only with autosync disabled

autosync also requires people to manually sync what is increasing chance of unsynced data, or data synced very late

In addition advanced setting were moved around - moving to top ones that are more likely to be usefully used
@westnordost
Copy link
Member

I have no data on this, but I believe that there are not few people who use this setting. Especially seasoned mappers who are used to other editors will prefer to upload and download manually (me included).

Anyway, there are two (more) legitimate reasons why this setting is important and not to be hidden in an advanced section, in my opinion:

  1. people with no mobile data flatrates
  2. very privacy-aware people since theoretically one could create movement profiles from your contributions in realtime

I too read all the reviews (in all languages) and I never identified autosync-off as a problem.

@westnordost westnordost added the feedback required more info is needed, issue will be likely closed if it is not provided label Jul 5, 2019
@matkoniecz
Copy link
Member Author

I have no strong feelings here and yes, this two uses are valid ones.

For reference, review that I mentioned was

Aplikacja ciekawa ale jak zaloguje się na swoje konto to nie można wysłać odpowiedzi. Po udzieleniu odpowiedzi ikona służąca do wysyłania wskazuje zero. Zwiększa się tylko liczba przy gwiazdce.

from https://play.google.com/store/apps/details?id=de.westnordost.streetcomplete&hl=pl&showAllReviews=true

it is complaining that upload is broken because count next to an upload icon is not increasing

Though on second thought it was fixed by interface change, as with autosync disabled it would not behave this way

@matkoniecz matkoniecz closed this Jul 5, 2019
@matkoniecz matkoniecz deleted the settings branch July 5, 2019 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feedback required more info is needed, issue will be likely closed if it is not provided
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants