Omar Jayed
Omar Jayed
> I have published a version of the package with the fix, you can use it via resolutions field in package.json > > ```json > { > "resolutions": { >...
This issue is over one year old. But there is no progress. Both OpenCensus and OpenTracing have merged into OpenTelemetry! So, it's better to move to OpenTelemetry.
Agree. A simple `ShutdownOption` should do that. For example `sd.Shutdown(fx.WithShutdownCause(err))`. What do you think?
I was testing my graphql resolvers. While calling the resolver, I got the panic. Maybe the field context is not present because I am calling the resolver manually. Anyway, the...
Thank @giautm! But surprisingly the empty string is working fine for me 😕 I will update it to use the edge name though!
I just pushed the changes! Now the generated code looks like: ```go func (o *Org) Members(ctx context.Context) ([]*User, error) { gqlAlias := "members" if fctx := graphql.GetFieldContext(ctx); fctx != nil...
Aha! Got it! I should test it like the way you mentioned instead. It would be better to add this to avoid panics in such scenarios. Still it's kinda unnecessary....
Thank @a8m, closing this 🙏
Hey there, is there any update on it?
Hey, it's been almost a year. Is there any update on this issue? Using this as a hotfix. correct me if I have done anything wrong. ```diff lc.Append(fx.Hook{ ++ OnStart:...