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.

Sign up for GitHub

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

Jump to bottom

docs should explain the @setupmethod behavior #4868

Closed
HLFH opened this issue Nov 16, 2022 · 3 comments
Closed

docs should explain the @setupmethod behavior #4868

HLFH opened this issue Nov 16, 2022 · 3 comments
Assignees
@davidism
Labels
docs

Comments

@HLFH
Copy link

HLFH commented Nov 16, 2022 •
edited by davidism

The appcontext documentation should explain the new @setupmethod 'shell_context_process' behavior, linked to #4577.

Flask raises an Assertion Error.

AssertionError: The setup method 'shell_context_processor' can no
longer be called on the application. It has already handled its
first request, any changes will not be applied consistently. Make
sure all imports, decorators, functions, etc. needed to set up the
application are done before running it.

The error should be documented, and a correct implementation should be provided.
@HLFH HLFH mentioned this issue Nov 16, 2022
Closed
@pallets pallets deleted a comment from HLFH Nov 16, 2022
@HLFH HLFH mentioned this issue Nov 17, 2022
Closed
@davidism davidism added the docs label Nov 17, 2022
@davidism davidism changed the title appcontext doc should explain the new @setupmethod 'shell_context_process' behavior docs should explain the @setupmethod behavior Nov 29, 2022
@bplaxco

This comment was marked as off-topic.

Sign in to view
@kemken077

This comment was marked as off-topic.

Sign in to view
@davidism davidism self-assigned this Feb 10, 2023
@davidism
Copy link
Member

Closed as part of #4973, see https://flask.palletsprojects.com/en/2.2.x/lifecycle/.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@davidism davidism

Labels
docs
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Explain AssertionError on the new @setupmethod 'shell_context_processor' behavior HLFH/flask
4 participants
@davidism @HLFH @bplaxco @kemken077