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

promote some quests before surface quest #1441

Merged
merged 2 commits into from
Jun 25, 2019

Conversation

matkoniecz
Copy link
Member

@matkoniecz matkoniecz commented Jun 25, 2019

according to my attempt at UX testing, some spotted reviews and interviewing some users
large number of surface quests may overhelm users, confuse them and result in failure
to discover other available quests

These two quests are more interesting to a mapper, except rare situation
will not be as repetitive as surface quest and except rare situation their number
of occurences will be far more limited.

This is intended to provide more interesting mapping for users.

Main negative is that in some places no quests of this type will be available, so
app will waste two Overpass requests and do not get quests


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

according to UX testing, some spotted reviews and interviewing some users
large number of surface quests may overhelm users, confuse them and fail
to discover other available quests

These two quests are more interesting to a mapper, except rare situation
will not be as repetitive as surface quest and except rare situation their number
of occurences will be far more limited.

This is intended to provide more interesting mapping for users.

Main negative is that in some places no quests of this type will be available, so
app will waste two Overpass requests and do not get quests.
@matkoniecz
Copy link
Member Author

I unfortunately failed to save links to this comments, but I may try to find again some of them.

@westnordost
Copy link
Member

'Tis reasonable 😃

@westnordost westnordost merged commit 818c17e into streetcomplete:master Jun 25, 2019
@matkoniecz matkoniecz deleted the quest_order branch June 30, 2019 14:27
@matkoniecz matkoniecz mentioned this pull request Jul 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants