Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

✏ Fix broken variable/typo in docs for Read Relationships, hero_spider_boy.id => hero_spider_boy.team_id #106

Merged
merged 4 commits into from
Aug 27, 2022

Conversation

yoannmos
Copy link
Contributor

Correcting the statement that select the team where Team.id == team.id of hero_spider_boy (1) not the hero.id (3 in this case)

yoannmos and others added 3 commits September 20, 2021 16:46
Correcting the statement that select the team where Team.id == team.id of hero_spider_boy (1) not the hero.id (3 in this case)
@tiangolo tiangolo changed the title 📖 Fix id => team_id ✏ Fix broken variable/typo in docs for Read Relationships, hero_spider_boy.id => hero_spider_boy.team_id Aug 27, 2022
@github-actions
Copy link

📝 Docs preview for commit f45d96e at: https://630a93f41b8fa54a81654f61--sqlmodel.netlify.app

@tiangolo
Copy link
Owner

Good catch, thank you! 🍰

I also fixed the relevant test. 🤓

@tiangolo tiangolo enabled auto-merge (squash) August 27, 2022 22:06
@tiangolo tiangolo merged commit ee576ab into tiangolo:main Aug 27, 2022
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.

None yet

2 participants