Global ids from objects with composite primary keys are not correctly extracted (again)

[genericmoniker]

In #43 a change was made to convert composite keys to a tuple.

graphql-core made a change such that a tuple is no longer accepted as a valid ID (whereas it would previously just call str on the tuple).

The result is an error such as the following when an object of the type is requested with its id field:

ID cannot represent value: (<UUID instance>, <UUID instance>)

[erikwrede]

Thanks for bringing this up! The easiest fix would most likely be to convert tuples to strings manually here:

graphene-sqlalchemy/graphene_sqlalchemy/types.py

Lines 382 to 385 in 2ca659a

	def resolve_id(self, info):
	# graphene_type = info.parent_type.graphene_type
	keys = self.__mapper__.primary_key_from_instance(self)
	return tuple(keys) if len(keys) > 1 else keys[0]

I'll check with the team to make sure this has no other implications and get back to you!

[erikwrede]

sorry for the long wait, this was fixed in #399

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related topics referencing this issue.