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
Check shop existence #4843
Check shop existence #4843
Conversation
fixes streetcomplete#4232 opening hours quest acts as a de facto checker of shop existence, but some people disabled it. separate from CheckExistence as very old shop with opening hours should show opening hours resurvey quest rather than this one (which would cause edit date to be changed and silence all resurvey quests)
app/src/main/java/de/westnordost/streetcomplete/quests/shop_type/CheckShopExistenceForm.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/de/westnordost/streetcomplete/quests/shop_type/CheckShopExistenceForm.kt
Outdated
Show resolved
Hide resolved
Co-authored-by: Flo Edelmann <git@flo-edelmann.de>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense! Useful for people who disabled the opening hours quest and adds very little code to maintain.
app/src/main/java/de/westnordost/streetcomplete/quests/shop_type/CheckShopExistence.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/de/westnordost/streetcomplete/quests/shop_type/CheckShopExistence.kt
Outdated
Show resolved
Hide resolved
Yeah, added for chance to implement something not growing into megaproject :) |
I'll approve this but not merge it yet because it is a new feature (albeit it does not require new strings). |
fixes #4232
Seems OK and plan on merging if noone will have feedback about this, but I want to give chance to review. No new text is being used here.
Why as a separate quest? Check existence quest is before opening hours, wheelchair quest and so on.
This one should be after opening hours etc, otherwise they would not be resurveyed in some cases. For example if check existence AND opening hours quests are enabled AND shop has
opening_hours
tagged AND shop node was not edited for 13 years, then opening hours resurvey quest should be asked, not check existence quest.Otherwise user would confirm existence, what would update object and opening hours resurvey would not be asked.
And why it is useful at all if opening hours already acts as a de facto existence checker? OH is quite time-consuming and some people could disable it without enabling wheelchair quest or similar.