dymmond / edgy

🔥 The perfect ORM to work with complex databases 🔥
https://edgy.dymmond.com
BSD 3-Clause "New" or "Revised" License
193 stars 9 forks source link

Feature:merged meta #79

Closed devkral closed 5 months ago

devkral commented 6 months ago

Changes:

Note: there should be also a small speed benefit by putting the model in the right registry in the first place

devkral commented 6 months ago

The reason why I use the clauses module as clausesmod is, that there are variable definitions of or or clauses

tarsil commented 5 months ago

This is great but:

  1. InspectDB generates edgy.ReflectModel type of models, so that needs updating to generate the new version of the reflect models.
  2. The documentation is not updated for this. There is a whole section about the reflect models. Bare in mind that it does not need to be removed, needs simply to be updated properly. It's an important section.

Inspect DB calls an object here:

https://github.com/tarsil/edgy/blob/main/edgy/utils/inspect.py#L274

devkral commented 5 months ago

I split out the deprecation fixes.

tarsil commented 5 months ago

@devkral as I mentioned before, just ignore what I said. I was missing something and you were correct. No worries I will have a proper look later but so far it seems ok 🙂