Skip to content

Can we change the name of ObjectMapperConfigurer? #234

Closed
@bsara

Description

@bsara

With the reorganization of code in the graph-java-... projects. There are now two ObjectMapperConfigurer classes. One in this project and one in graphql-java-tools. It seems like it would be prudent to rename them each to be specific to their respective projects...that, or share the code somehow between them. I just got bit by the fact that I thought I was using the correct ObjectMapperConfigurer because I thought that there was only one...only to finally discover, after a few hours of debugging, that there are actually two classes with the same name.

Perhaps we could name them GraphQLServletObjectMapperConfigurer and GraphQLSchemaParserObjectMapperConfigurer?

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions