Georg Semmler

Thanks for opening this PR. Generally speaking it is fine to add such an impl, but we cannot accept a `SqlOrder` impl that is conditionally applied if a specific feature...

Thanks for opening this PR. Can you add a description what the change want to achieve?

I'm definitively open to include changes required to enable more third party backends. Its fine to open a test PR here, but in general I would prefer to have an...

@alex0ide Not really as it works if `date` in that example has the "right" sql type (`timestamp`, not `timestamptz`.) As for the original example: The following query works and generates...

@Fuuzetsu That's not really this issue. It's the same underlying problem, but it can be fixed much more easily. Instead of just accepting a `Timestamp` type [here](https://github.com/diesel-rs/diesel/blob/3f3ce57ca1a89274ecf915b70826234c45bd261e/diesel/src/expression/functions/date_and_time.rs#L46) that could be...

@mikhail-krainik That was [this](https://github.com/rust-lang/crates.io/blob/7a52b47781cacdf88064a578f4f7cb4e3cf015e4/src/krate.rs#L105) thing. But be warned: I'm quite sure doing that is a bad idea…

@dyst5422 I've written [wundergraph](https://github.com/weiznich/wundergraph) for this use case as you have to think about much more than a few field attributes for providing a graphql API that does not suffer...

@tuxzz Just commenting here that you need a certain feature will help that the feature appears in a magical way. You either need to implement all traits manually, which would...

@alfonso-eusebio As a general note at the top: As pointed out in this comment (https://github.com/diesel-rs/diesel/issues/860#issuecomment-300280659) and this PR (https://github.com/diesel-rs/diesel/pull/1582) we are not interested in having an option to ignore fields...

@alfonso-eusebio There is no need to apologies here. Anyone that comes with a well written proposal as yours is well come to ask questions and propose improvements. @lolgesten > Which...