Kris Lefeber

Results 6 comments of Kris Lefeber

I'm not sure of the proper way to handle this, but I was able to resolve it by linking to the actual module.ts file under the node_modules folder. It was...

Is it null in both the constructor and in generateDataLoader()? It looks like context is not readonly, which could prevent it from being an instance variable. That's a javascript thing,...

I don't believe that would fit with the design of dataloader. The intention is that you already have an ID that ties to a specific value in some data store....

@gouroujo, if we can get typing added to the passed in context variable I will go ahead and make the merge. I see my mistake about the optional parameter

I created a small npm package, nestjs-dataloader, since I also struggled with this. Hopefully it helps someone else.

What is your use case for this? If you don't want it to auto-generate, then don't use the PrimaryGeneratedColumn. If you manually set this id, the auto-increment could break on...