Weaved schema doesn't include type descriptions

[adamkl]

I just spent a little time updating the type descriptions on one of by back-end graphql services and I noticed that these descriptions don't appear to be displayed in the weaved schema on my gateway server.

I'm looking to provide a self serve graphql API to my consumers so these type descriptions would be pretty valuable.

I'm going to see if I can get some time to work on this myself if you're open to pull requests.

[Yogu]

Thanks for the bug report. The most obvious source would be if I forgot it here, but that's not the case. I don't know where else the problem could be off the top of my head. I'll see if I have time to try to reproduce the bug in the next days. If you have the time, we are definitely open for pull requests. 🎉

[adamkl]

I'll double and triple check on my end to make sure I'm not wasting your time.

[Yogu]

It might very well be a bug, there are some other places where type configs are partially changed. Maybe you can check for which kind of types it applies? E. g. The root query type, normal types, such types involved in link/join, or all of them?

…

On Mo., 25. Sep. 2017, 20:00 adamkl ***@***.***> wrote: I'll double and triple check on my end to make sure I'm not wasting your time. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#7 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAYKIgST87uFL5TRqon7idFqCqXaDWabks5sl-o-gaJpZM4PjEWy> .

[adamkl]

Before anyone else wastes any time looking into this, with my latest config/deploy, I can see that the descriptions are being weaved correctly.

I removed the namespace/typePrefix from my configuration and the descriptions appeared. I'm going to play around a bit more to see if I can repro the issue.

[adamkl]

I haven't been able to repo this. You should probably close this one.