Support for Decimal type

[picturedots]

I've been using my own implementation of a Decimal type -- maybe it should be included as a standard type?

from graphene.types import Scalar
from graphql.language import ast

class Decimal(Scalar):
    """
    The `Decimal` scalar type represents a python Decimal.
    """
    @staticmethod
    def serialize(dec):
        assert isinstance(dec, decimal.Decimal), (
            'Received not compatible Decimal "{}"'.format(repr(dec))
        )
        return str(dec)
    
    @staticmethod
    def parse_value(value):
        return decimal.Decimal(value)
    
    @classmethod
    def parse_literal(cls, node):
        if isinstance(node, ast.StringValue):
            return cls.parse_value(node.value)

[jkimbo]

Great idea! Would love to see a PR with this and some tests.

Also parse_value should probably return None if it's not a valid Decimal (like how it's done in the datetime types:

graphene/graphene/types/datetime.py

Lines 60 to 64 in 05d7e61

	def parse_value(value):
	try:
	return parse_datetime(value)
	except ValueError:
	return None

)

[picturedots]

Ok, I will take a crack at it when I have a chance. I'll have a look at the existing tests for other types for some examples.

[jkimbo]

Decimal support has now been merged into master. Thanks @picturedots !

[iraj-jelo]

It's cool guys. I came across this issue now and I found I should update the Graphene to get new fields. actually I used to check the document for new features but apparently it isn't update though.

[jkimbo]

@iraj-jelo the documentation is a bit out of date 😞 We are accepting pull requests though if you'd like to update it?

[jkimbo]

(also it is there in the API documentation at least! https://docs.graphene-python.org/en/latest/api/#graphene-scalars)