generated client entity projection missing property for extended entity about dgs-codegen HOT 9 CLOSED

Hi @paulbakker.

My repo is here https://github.com/deweyjose/graphqlcodegen. The plugin is already available in maven central.
Also a very thin example on how to use it https://github.com/deweyjose/graphqlcodegen-example.

I'm happy to help out if possible, but I'm not exactly sure how to build a maven plugin from a gradle build :).

from dgs-codegen.

Are you using the latest release of the plugin - v4.1.3? We did recently fix an issue around generation of projections when the schema has cycles (which it does in your case).
I am seeing the projection being generated with the expected fields in the latest code.

from dgs-codegen.

We're actually using 4.1.2. I've built a Maven port of the codegen plugin (most of our projects are maven based) and 4.1.3 of the codegen core hasn't been released yet. Once it is I'll integrate it. Thanks so much!

Related to that - any plans on a maven plug-in? I'd be more than happy to contribute back if interested. It's not that much code at all.

from dgs-codegen.

Great, let us know how it works out.
Regarding Maven, we've actually had a few requests for it since we open sourced the framework and plugin. We don't have any demand internally for it and hence haven't prioritized it. We do welcome code contributions, so if you have something already working, that would be great!

from dgs-codegen.

We're actually using 4.1.2. I've built a Maven port of the codegen plugin (most of our projects are maven based) and 4.1.3 of the codegen core hasn't been released yet. Once it is I'll integrate it. Thanks so much!

Related to that - any plans on a maven plug-in? I'd be more than happy to contribute back if interested. It's not that much code at all.

A contribution would be much appreciated! I would expect it's not that much code (the gradle plugin is pretty thin as well), but it's mainly figuring out how maven plugins work and testing that's holding us back from actually doing it.

from dgs-codegen.

@deweyjose This is great!

If you're ok with it, I would love to reference this in the docs.
As for the repository itself, as you already mentioned, I'm not sure if we can technically put the plugins in the same repo because this needs to be a Maven based repository. So we could just leave the plugin where it is for now?

from dgs-codegen.

@paulbakker absolutely. I plan on adding some tests to the plug-in repo as well as add breadth to my example repo.

from dgs-codegen.

@deweyjose - is your original issue fixed after trying the latest version of the plugin? Can we close this issue?

from dgs-codegen.

Closing this issue for now, please open a new issue if the problem persists.

from dgs-codegen.