Skip to content

Conversation

GeigerJ2
Copy link
Contributor

@GeigerJ2 GeigerJ2 commented Sep 4, 2025

Copy link
Contributor

@agoscinski agoscinski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

footnotes do not render correctly for me

│ User interface │ ← Node (Python ORM class)
│ (orm/nodes) │
├─────────────────────┤
│ Backend interface │ ← BackendNode (Abstract base class)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't full understand it but backend interface is not a good name since you also sometimes refer to the database as backend. Is this like a orm node backend?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, yes, good point! It's basically the abstract base class BackendNode.
I changed it to abstract interface, as I also find the name BackendNode a bit confusing (we are talking about many backends overall, db backend, ORM backend, etc. ^^)
Do you agree that this is clearer? Overall, in my last commit, I modified the boxes and section headings. Should be much clearer now (at least I think so).

@GeigerJ2 GeigerJ2 merged commit 78ad9c4 into main Sep 5, 2025
3 checks passed
@GeigerJ2 GeigerJ2 deleted the blog/orm branch September 5, 2025 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants