-
-
Notifications
You must be signed in to change notification settings - Fork 365
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
Always ask if defibrillator (AED) is indoor #5336
Comments
Hmm, so why for defibrillators specifically? What is this property AEDs have that it makes sense to always ask for their location? Could other POIs have this property, too? |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
Maybe the property I am looking for would be "is usually found on walls (of a building)". A category to which ATMs may belong to, too? Or are there many fee-standing ones too, as there are vending machines in general that may also be free-standing? |
I think for such important (livesaving) things like AED, it's ok if we ask it always, but for atms this might be a bit too spammy. |
Asking if it is indoor or outdoor is much faster (one click) than entering a description of the precise location, so it probably will be answered more often. And since it may be life saving to know whether the AED is indoor, even if the exact location was not specified, I think that it is worth the exception. Also, the location description quest can be answered more easily when there is an "indoor" or "outdoor" location label above; otherwise we might risk that people just answer e.g. "outdoor" in the free text field. |
Use case
In #5278, the
AddIsDefibrillatorIndoor
quest was replaced with the more genericAddIsAmenityIndoor
quest. The old quest was asked for allemergency = defibrillator
nodes, while the new quest only asks about nodes inside a building outline. E.g. for node 11071807905 (see also this node in OpenAEDMap), the new quest is not shown.It would be good to relax that filtering in the new quest for AEDs again. See also the original reasoning by @matkoniecz in #2068:
Proposed Solution
Keep the new quest, but always show the quest for
emergency = defibrillator
nodes, even if they are not inside a building outline.cc @qugebert
The text was updated successfully, but these errors were encountered: