-
-
Notifications
You must be signed in to change notification settings - Fork 339
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
Extract quest data models #2501
Extract quest data models #2501
Conversation
BikeParkingType
data model
app/src/main/java/de/westnordost/streetcomplete/quests/kerb_height/KerbHeight.kt
Show resolved
Hide resolved
// sorted by ~worldwide usages, minus country specific ones | ||
MULTI("multi"), | ||
|
||
// 250k - 10k |
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 comments about the usage numbers should actually go to the quest form, because this is where the order matters. The order does not matter in an enum (read: should 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.
But I can just do that after the merge
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.
In the quest form, the list is sorted by country usage anyway, so order there shouldn't matter either.
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.
only the top few
🚀 |
So, all fine, I'll merge it. Thank you for your work! |
Follow-up to #2495. Fixes second part and closes #2494.