Skip to content
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

graphql-workshop-neo4j serialization issue on Code First #5719

Closed
1 task done
bnayae opened this issue Jan 26, 2023 · 1 comment
Closed
1 task done

graphql-workshop-neo4j serialization issue on Code First #5719

bnayae opened this issue Jan 26, 2023 · 1 comment
Labels
Area: Neo4J Issue is related to the Neo4J integration 🌶️ hot chocolate

Comments

@bnayae
Copy link

bnayae commented Jan 26, 2023

Is there an existing issue for this?

  • I have searched the existing issues

Product

Hot Chocolate

Describe the bug

graphql-workshop-neo4j serialization issue on Code First

Unless I'm naming my properties as a camel-case in the first place, results return as nulls.

image

Even when I tried to use [JsonPropertyName("title")], it didn't help.

    public class Movie
    {
        [JsonPropertyName("title")]
        public string Title { get; set; }

        [Neo4JRelationship("ACTED_IN", RelationshipDirection.Incoming)]
        public List<Actor> Actors { get; set; }
    }

The only thing which work for me is using a camel case (which is against our convention).

    public class Movie
    {
        public string title { get; set; }

        [Neo4JRelationship("ACTED_IN", RelationshipDirection.Incoming)]
        public List<Actor> Actors { get; set; }
    }

I expect the library being able to use Pascal Case properties and honor [JsonPropertyName("title")].

Steps to reproduce

  1. Just run the code first sample of the workshop
    https://github.com/ChilliCream/graphql-workshop-neo4j

You can change the project setting to

    <PropertyGroup>
		<TargetFramework>net7.0</TargetFramework>
		<LangVersion>11.0</LangVersion>
		<Nullable>disable</Nullable>
		<ImplicitUsings>enable</ImplicitUsings>
	</PropertyGroup>

    <ItemGroup>
      <PackageReference Include="HotChocolate.AspNetCore" Version="12.16.0" />
      <PackageReference Include="HotChocolate.Data.Neo4J" Version="12.16.0" />
    </ItemGroup>

Yet, it doesn't work on the original either.

Relevant log output

No response

Additional Context?

No response

Version

12.16.0

@tobias-tengler tobias-tengler added Area: Neo4J Issue is related to the Neo4J integration 🌶️ hot chocolate labels Jan 26, 2023
@glen-84
Copy link
Collaborator

glen-84 commented Jan 16, 2024

Closing, as Neo4j support has been removed in v14.

@glen-84 glen-84 closed this as not planned Won't fix, can't repro, duplicate, stale Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Neo4J Issue is related to the Neo4J integration 🌶️ hot chocolate
Projects
None yet
Development

No branches or pull requests

4 participants