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

📝 Add warning about lifespan functions and backwards compatibility with events #10734

Merged
merged 4 commits into from Jan 12, 2024

Conversation

jacob-indigo
Copy link
Contributor

Backwards compatibility with startup/shutdown events in implemented in Starlette with a default lifespan function that calls the registered event handlers. If there is a user-provided lifespan function, those event handlers will no longer be called.

@tiangolo
Copy link
Owner

📝 Docs preview for commit a117d0e at: https://a824f295.fastapitiangolo.pages.dev

@tiangolo
Copy link
Owner

tiangolo commented Dec 4, 2023

📝 Docs preview for commit d432e56 at: https://c6dbf0de.fastapitiangolo.pages.dev

@jacob-indigo
Copy link
Contributor Author

@tiangolo This is my first time contributing to FastAPI. Is there anything else I need to do for this PR?

@tiangolo
Copy link
Owner

📝 Docs preview for commit df51abb at: https://e35849a0.fastapitiangolo.pages.dev

@tiangolo
Copy link
Owner

tiangolo commented Jan 8, 2024

📝 Docs preview for commit 19db41c at: https://1d141478.fastapitiangolo.pages.dev

@alejsdev alejsdev changed the title ✏️ docs: Add warning about lifespan functions and backwards compatibility with events 📝 Add warning about lifespan functions and backwards compatibility with events Jan 10, 2024
@alejsdev alejsdev added the docs Documentation about how to use FastAPI label Jan 10, 2024
@tiangolo
Copy link
Owner

Good idea, thank you for your contribution @jacob-indigo! 🍰 ☕

@tiangolo tiangolo merged commit 7e0e16f into tiangolo:master Jan 12, 2024
37 checks passed
nilslindemann added a commit to nilslindemann/fastapi that referenced this pull request Jan 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation about how to use FastAPI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants