DerivePartialModel: Allow specification of Entity via a Path in addition to as an Ident

jreppnow commented 4 months ago

PR Info

Content: With the current implementation when deriving DerivePartialModel, one must specify the Entity whose columns should be used as an Ident - meaning that one must import it into the current scope
This is somewhat unwieldy, especially when dealing with generated code, where one would prefer to just use the containing module, not the contents themselves
This PR instead allows one to specify a Path, meaning that no direct use is required anymore
Ticket: Sorry, should have probably created a ticket first - does not exist atm, but can create one if necessary.

[x] allow the Entity to be specified as a Path, not just an Ident, alleviating the need to import the Entity struct whenever you want to use the DerivePartialModel macro

The changes should be backwards-compatible/a pure extension, since Paths are a superset of Idents

[x] fixed a typo in the changed file ("specific" to "specified") in a couple of places

tyt2y3 commented 3 months ago

I think the intent of this PR is mostly already covered by https://github.com/SeaQL/sea-orm/pull/2137 If you have something to amend, please rebase onto master, thanks!

jreppnow commented 3 months ago

@tyt2y3 Yup, that one pretty much covers it! Sorry for taking a bit longer to respond, I will close this PR as it's been superseded.