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

Add amenity driving_school in AddOpeningHours quest #4899

Closed
bmaggi opened this issue Mar 23, 2023 · 3 comments
Closed

Add amenity driving_school in AddOpeningHours quest #4899

bmaggi opened this issue Mar 23, 2023 · 3 comments

Comments

@bmaggi
Copy link
Contributor

bmaggi commented Mar 23, 2023

Use case

  • when mapping StreetComplete should (IMHO) propose to add opening hour to driving school (In France, driving_school propose theoretical training and have opening hours for that)

Proposed Solution

⚠️ This issue suggest that at some point (old java version) driving_school was already present in this quest (but it's not present at the start of the kotlin file)

I can push a MR if this change is accepted

@matkoniecz
Copy link
Member

I would say that adding it back would make sense, as far as I can see, seems to be accidentally gone when converting to Kotlin. It may be worth checking whether anything else got lost in opening hours quest.

Feel free to make PR, just test whether it works. Thanks for spotting it!

bmaggi added a commit to bmaggi/StreetComplete that referenced this issue Mar 23, 2023
…ours quest

amenity driving_school was lost in the migration from java to kotlin
bmaggi added a commit to bmaggi/StreetComplete that referenced this issue Mar 23, 2023
…ours quest

amenity driving_school was lost in the migration from java to kotlin
@bmaggi
Copy link
Contributor Author

bmaggi commented Mar 23, 2023

Here is the PR. I didn't test locally for the moment (I have to reinstall the android studio since my version is too old, will try to do it tomorrow)

Note that "horse_riding" also disappeared without explicit explanation but was probably included in this comment L 76
// not sports_centre, dance etc because these are often sports clubs which have no
// walk-in opening hours but training times

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

No branches or pull requests

2 participants