-
-
Notifications
You must be signed in to change notification settings - Fork 358
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
house name and house number answer at once #3582
Conversation
Add another answer "House number and a name" which gives a two field form, for house name and house number. fixes streetcomplete#2540
Awesome, any thoughts on how SC could help capture those housenames not captured by SC originally because the mapper could only enter the number? |
app/src/main/java/de/westnordost/streetcomplete/quests/address/AddHousenumberForm.kt
Outdated
Show resolved
Hide resolved
Currently: only solution is to wait for resurvey. I can't think of anything which could be done for this specifically, but in general, either the layers feature or my proposed alternative, "bulk edit mode" (#2461 (comment)) would allow editing at times when a quest would not normally be shown. |
This is likely not feasible to do as a quest. Asking for housenumber for all buildings with house name where someone answered with partial info instead of making note is not a good idea as vast majority has just housename. Unless in some regions it is safe to assume that building always has house number and house name is a possible addition. |
app/src/main/java/de/westnordost/streetcomplete/quests/address/AddHousenumberForm.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/de/westnordost/streetcomplete/quests/address/AddHousenumberForm.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/de/westnordost/streetcomplete/quests/address/AddHousenumberForm.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/de/westnordost/streetcomplete/quests/address/AddHousenumberForm.kt
Outdated
Show resolved
Hide resolved
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.
Typo
app/src/main/java/de/westnordost/streetcomplete/quests/address/AddHousenumberForm.kt
Outdated
Show resolved
Hide resolved
I don't believe there is a resurvey and don't see any code for one either. But I can think of very few cases when a house number or name would change either.
There is some potentially valid tagging for it: But aside from asking for every house as others have said I can't think of a sensible way to filter it. |
idea by FloEdelmann
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.
Just an observation, this quest includes:
Housenumber
HouseNumber
HOUSENUMBER
House number
And similar for house name.
I appreciate the OSM key is addr:housenumber and the English is "House Number", but would it be worth being consistent and either using housenumber/HOUSENUMBER/Housenumber throughout, or treating it as two words everywhere apart from when it's the actual OSM key (i.e. houseNumber/HOUSE_NUMBER/HouseNumber).
Likewise, but not as bad, for house name.
Co-authored-by: smichel17 <github@smichel.me>
29abe71
to
1632c30
Compare
@smichel17 @FloEdelmann @peternewman Thanks for reviews here and elsewhere! |
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.
Alright, also review by me. A few things could be simplified. There is one UI issue I see (IMO).
app/src/main/java/de/westnordost/streetcomplete/quests/address/AddHousenumberForm.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/de/westnordost/streetcomplete/quests/address/AddHousenumberForm.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/de/westnordost/streetcomplete/quests/address/AddHousenumberForm.kt
Outdated
Show resolved
Hide resolved
Co-authored-by: Tobias Zwick <osm@westnordost.de>
234d938
to
2fccf63
Compare
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.
Looks good to me.
app/src/main/java/de/westnordost/streetcomplete/quests/address/AddHousenumberForm.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/de/westnordost/streetcomplete/quests/address/AddHousenumberForm.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/de/westnordost/streetcomplete/quests/address/AddHousenumberForm.kt
Outdated
Show resolved
Hide resolved
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.
Looks good to me now. Thanks @matkoniecz for implementing the feedback!
Add another answer "House number and a name" which gives a two field form, for house name and house number.
fixes #2540
Tested case where housenumber is entered, form switched to housename - it record just housename as it should
NOTE: PR code needs one filler string in unrelated quest to compile