You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, the _edge_definitions specify the collections as strings. To be able to use introspection correctly, it should be possible to specify the class of the collection directly instead. I would make it as a backwards compatible change. Example:
Currently you often see the example with CollectionA._name. Accessing a protected variable in this way also leads to a warning in the IDEs. We should try to be PEP8 compatible if possible.
Furthermore, this change allows a good solution for #24, where I have to introspect _edge_definitions.
The text was updated successfully, but these errors were encountered:
Currently, the _edge_definitions specify the collections as strings. To be able to use introspection correctly, it should be possible to specify the class of the collection directly instead. I would make it as a backwards compatible change. Example:
Currently you often see the example with
CollectionA._name
. Accessing a protected variable in this way also leads to a warning in the IDEs. We should try to be PEP8 compatible if possible.Furthermore, this change allows a good solution for #24, where I have to introspect _edge_definitions.
The text was updated successfully, but these errors were encountered: