-
Notifications
You must be signed in to change notification settings - Fork 2
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
more abstract geojson approach #173
Conversation
src/parkapi_sources/converters/base_converter/pull/static_geojson_mixin.py
Outdated
Show resolved
Hide resolved
@@ -112,7 +111,8 @@ class HerrenbergBikePropertiesInput(ValidataclassMixin): | |||
operator_name: OptionalUnset[str] = NoneToUnsetValue(StringValidator(min_length=0, max_length=256)), DefaultUnset | |||
capacity: int = IntegerValidator(min_value=0) | |||
capacity_charging: OptionalUnset[int] = NoneToUnsetValue(IntegerValidator(min_value=0)), DefaultUnset | |||
capacity_cargobike: OptionalUnset[int] = NoneToUnsetValue(IntegerValidator(min_value=0)), DefaultUnset | |||
# capacity_cargobike is unsupported yet |
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.
Question: is it known when it will be supported? I am wondering if it would help to write the comment as a TODO + number of the issue where it will be fixed, so it's easier to find it later.
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.
Did not do a TODO, as it is no TODO atm, but linked the ticket: #174
9c053a6
to
5e514df
Compare
As #168 was running into strange errors during testing, I made the GeoJSON approach a bit more abstract and moved it to a general model.