- 
          
- 
                Notifications
    You must be signed in to change notification settings 
- Fork 784
          📝 Update FastAPI tutorial docs to use the new model.sqlmodel_update() instead of old setattr()
          #1117
        
          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
Conversation
| 📝 Docs preview for commit 507b5fd at: https://ca1eec8a.sqlmodel.pages.dev Modified Pages | 
| 📝 Docs preview for commit b1f3f90 at: https://d6ac7646.sqlmodel.pages.dev Modified Pages | 
| @tiangolo now it is all green with your last commits :) | 
| 📝 Docs preview for commit 99b68d3 at: https://2dfc2c64.sqlmodel.pages.dev Modified Pages | 
| 📝 Docs preview for commit 804c693 at: https://8304b488.sqlmodel.pages.dev Modified Pages | 
| 📝 Docs preview for commit 6e94e6f at: https://9f0e2f11.sqlmodel.pages.dev Modified Pages | 
model.sqlmodel_update() instead of old setattr()
      | 📝 Docs preview for commit 072a99e at: https://8b042c1a.sqlmodel.pages.dev Modified Pages | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, thank you! 🚀
I ran the local docs dev server to check the docs, I checked the tutorial for FastAPI to check the highlights and updated the ones in docs/tutorial/fastapi/teams.md, it seems those were the only ones that needed updating, the rest seemed fine.
Thank you! ☕
change old setattr() for sqlmodel.sqlmodel_update(), is has been updated in version 0.0.16
also in contributing, the name of the scripts to execute the test and the coverage has changed the name.