nexus-plugin-prisma icon indicating copy to clipboard operation
nexus-plugin-prisma copied to clipboard
verified publisher icon graphql-nexus

Fix/remove crud pluralize

Open alanstriglio opened this issue 5 years ago • 3 comments

This branch fix the pluralize issue #537

Instead of Query t.crud.user and t.crud.users the new methods are now t.crud.findOneUser and t.crud.findManyUser following the Mutation methods style createOneUser.

I know this is a great change for current user, so take your time to evaluate this PR.

Also I fix some tests that was broke/broken.

alanstriglio avatar Aug 25 '20 21:08 alanstriglio

What do you think about using pluralization after findMany? So findManyUser becomes findManyUsers?

P4sca1 avatar Jan 12 '21 01:01 P4sca1

That approach would solve the problem just as well, but it wouldn't be in line with the prisma.io project.

alanstriglio avatar Jan 18 '21 07:01 alanstriglio

Any idea when this will be merged 🙏?

nealoke avatar Oct 13 '21 06:10 nealoke