Tycho Bokdam

Results 140 comments of Tycho Bokdam

Have not looked at it, `__` think it's good, changes of that colliding with other fields should be pretty low.

If you add the field `someExtraField` to the entity without the `@Column` does it than work?

Sorry for the late reply, completely missed this! The issue is very weird as when you extend `CRUDResolver` it will be the same as defining `resolvers` inside your module. I...

You also have this issue if you also provide a `count` / `sum`?

Will see if I can try to reproduce this, there is not anything special that you are doing? As, in my own projects everything works and all tests also passed.

I think the reason for this change was to support grouping by week/day/month, commit: 7ffeaf6b9e400eb027298a3870712eb7124c88bb, we can check to update this to the old behaviour for non date fields.

@Flusinerd would it be possible for you to tryout something? Could you update `src/query/aggregate.builder.ts` the constructor to always set `this.isPostgres = true`, I think it should work than. If this...

Sorry after checking a bit more the change is not going to make a difference. I do now see the change what caused it, when implementing the grouping of dates...

Could be interesting to add! But to much work for myself as I also don't use / have a project using Prisma.

Interesting @dvins, I myself now also started using Prisma in a project and wonder: would something basic not work if we only go based on the metadata nestjs-query itself collects...