Converters cannot convert datetime.timedelta to float.

[dionyself]

hello devs.
During a conversation about an issue of graphql-core @jkimbo found a possible error

graphene-django/graphene_django/converter.py

Lines 111 to 113 in 5051d3b

	@convert_django_field.register(models.DurationField)
	def convert_field_to_float(field, registry=None):
	return Float(description=field.help_text, required=not field.null)

raising something like { "message": "float() argument must be a string or a number, not 'datetime.timedelta'" }

you can check graphql-python/graphql-core#150 for more info.

[spockNinja]

Yeah, I can see how that'd be a problem... Even if it could convert, a float doesn't quite capture all of the data in a timedelta.

One possibility would be to add an ObjectType wrapper that pulls out the individual parts of the timedelta and send that, potentially with one property being the "human readable" representation.

Any other ideas?

[dionyself]

I managed to get an human readlable for string type by using duration = graphene.String()

https://github.com/dionyself/leaky/blob/71ec608204a3dd66f2fcd25ff36096ba25eb62bc/warehouses/schema/query/asset.py#L8-L13

We probably won't need and human readable for the Float converter, so using timedelta.total_seconds() is fine... but, i think this works in py3 only

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[zeth]

This is a bad bot. This is still an issue.

[jkimbo]

Apologies @zeth , reopening the issue. If you have any time any help with the issue would be most appreciated.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.