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

Update reqcontext.rst #4267

Merged
merged 1 commit into from
Oct 1, 2021
Merged

Update reqcontext.rst #4267

merged 1 commit into from
Oct 1, 2021

Conversation

sprutner
Copy link
Contributor

Fixing a grammar error.

  • fixes #

This is just a typo correction.

Checklist:

  • Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
  • Add or update relevant docs, in the docs folder and in code.
  • Add an entry in CHANGES.rst summarizing the change and linking to the issue.
  • Add .. versionchanged:: entries in any relevant code docs.
  • Run pre-commit hooks and fix any issues.
  • Run pytest and tox, no tests failed.

@davidism davidism closed this Sep 21, 2021
@davidism
Copy link
Member

"an AppContext" is correct

@sprutner
Copy link
Contributor Author

sprutner commented Sep 22, 2021 via email

@davidism
Copy link
Member

Oh I see the one you were referring to now. Looks like there's a few other links with that issue, please address those as well.

@davidism davidism reopened this Sep 22, 2021
@davidism
Copy link
Member

davidism commented Oct 1, 2021

I went through and fixed all the other uses of app and request context.

@davidism davidism added the docs label Oct 1, 2021
@davidism davidism merged commit f8c881b into pallets:2.0.x Oct 1, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants