Skip to content
This repository has been archived by the owner on Dec 31, 2023. It is now read-only.

fixed some file #8

Closed
wants to merge 14 commits into from
Closed

fixed some file #8

wants to merge 14 commits into from

Conversation

miss85246
Copy link
Contributor

  1. .gitignore add package-lock.json
  2. Unified subtitle level, set to h4 in basics/middleware.md
  3. fixed spell error: endpoing -> endpoint in basics/routing.md

@miss85246
Copy link
Contributor Author

@ahopkins In the process of translation, I found several places that need to be modified, and I made simple modifications.

@ahopkins
Copy link
Member

ahopkins commented Feb 2, 2021

Yes, there is still lots of work to be done. The goal is to remove that BETA notice at the bottom simultaneously with the release of v21.3 which will include a PR to remove the docs from RTD.

@miss85246 miss85246 marked this pull request as ready for review February 2, 2021 10:46
@@ -104,7 +104,7 @@ async def halt_response(request, response):
```
:---

## Order of execution
#### Order of execution
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why this and not H2?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

look at this file : basics/headers.md Tokens and Proxy headers are part of Headers and they are H4 title.
In like manner, this also should be H4 title

@miss85246 miss85246 closed this Feb 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants