-
-
Notifications
You must be signed in to change notification settings - Fork 394
Charging station quest #6531
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
base: master
Are you sure you want to change the base?
Charging station quest #6531
Conversation
Added and filled the backlog for first sprint
…lete#6457, the icons of the quest need to be updated
|
very quick comment
this is not needed - translations are pulled from translation service and override previous contents, see https://github.com/streetcomplete/StreetComplete/blob/master/CONTRIBUTING.md#translating-the-app
|
app/src/commonMain/kotlin/de/westnordost/streetcomplete/ApplicationConstants.kt
Outdated
Show resolved
Hide resolved
|
|
||
| override val elementFilter = """ | ||
| nodes, ways with | ||
| amenity = charging_station |
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.
I think we should also filter out any charging stations with motor_vehicle=yes/designated since some charging stations can be used for both scooters / bikes and cars.
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.
Hm well, I guess the situation with these vehicle tags is a bit difficult and/or undocumented.
Is a amenity = charging_station and scooter ~ yes|designated supposed to be a scooter-only charging station? Or are motor vehicles generally allowed unless specified otherwise? The wiki makes it sound like it is supposed to be the latter.
On the other hand, unless specified explicitly in the wiki (and even then), we have to assume that we'll find both in the wild.
I am thinking that maybe it would be safer to always tag capacity:scooter for clarity because amenity=charging_station is . (Same for bicycles)
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.
Could we not reuse the solution in #6536?
Tag capacity:scooter if the charging station supports other modes of transport and capacity if not?
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.
The problem is that the list of "other modes of transport" is somewhat open ended.
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.
My message is the duplicate of one I made below.
@westnordost I'm don't really see the problem about the "other modes of transport" behing open ended. A charging station with multiple vehicule types (e.g. bicycle, car and scooter) might have informations about the bicycle and car's capacity but not the scooter's.
| override val elementFilter = """ | ||
| nodes, ways with | ||
| amenity = charging_station | ||
| and bicycle ~ yes|designated |
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.
Same here
app/src/androidMain/kotlin/de/westnordost/streetcomplete/quests/QuestsModule.kt
Outdated
Show resolved
Hide resolved
|
Nice work so far! |
|
I also think the quests are better placed together with the car one in quests/charging_station_capacity instead of a new directory. |
|
|
||
| override val elementFilter = """ | ||
| nodes, ways with | ||
| amenity = charging_station |
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.
Hm well, I guess the situation with these vehicle tags is a bit difficult and/or undocumented.
Is a amenity = charging_station and scooter ~ yes|designated supposed to be a scooter-only charging station? Or are motor vehicles generally allowed unless specified otherwise? The wiki makes it sound like it is supposed to be the latter.
On the other hand, unless specified explicitly in the wiki (and even then), we have to assume that we'll find both in the wild.
I am thinking that maybe it would be safer to always tag capacity:scooter for clarity because amenity=charging_station is . (Same for bicycles)
...streetcomplete/quests/scooter_charging_station_capacity/AddScooterChargingStationCapacity.kt
Outdated
Show resolved
Hide resolved
...rdost/streetcomplete/quests/bike_charging_station_capacity/AddBikeChargingStationCapacity.kt
Outdated
Show resolved
Hide resolved
I don't mind either way. Don't see a reason for that. |
If #6536 gets merged there will be shared code that all quests might use in that directory |
app/src/commonMain/kotlin/de/westnordost/streetcomplete/ApplicationConstants.kt
Outdated
Show resolved
Hide resolved
…ationConstants.kt Co-authored-by: Mateusz Konieczny <[email protected]>
…lter and tag to look for capacity:scooter/bicycle instead of capacity. Adding an icon for the quests. Removing empty lines from QuestsModule.
...ost/streetcomplete/quests/charging_station_capacity/AddScooterChargingStationCapacityForm.kt
Show resolved
Hide resolved
|
|
||
| override val elementFilter = """ | ||
| nodes, ways with | ||
| amenity = charging_station |
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.
The problem is that the list of "other modes of transport" is somewhat open ended.
|
@westnordost I'm don't really see the problem about the "other modes of transport" behing open ended. A charging station with multiple vehicule types (e.g. bicycle, car and scooter) might have informations about the bicycle and car's capacity but not the scooter's. |
|
So... I'm not sure what to do now @westnordost @paulklie. Is my pull request correct ? Do I need to resolve the 2 remaining conflicts even if no modifications were made ? |
Adding the following quests suggested in #6457:
For now, the icon of the quest 2 form is a bike and need to be replaced with a scooter. The icons of both quests are the same as the original charging_station_capacity quest, they could be replaced.
This PR include a french translation of the quests.