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 caching based on RFC9111 #16009

Merged
merged 34 commits into from
May 16, 2022
Merged

Update caching based on RFC9111 #16009

merged 34 commits into from
May 16, 2022

Conversation

Jxck
Copy link
Contributor

@Jxck Jxck commented May 13, 2022

Summary

It's been a long since #10027 but it's the rest of task updating caching section align with RFC9111 (new RFC not published yet).

Motivation

updating http caching section align with cache-control section and new RFC9111

Related issues

#8668

Metadata

  • Adds a new document
  • Rewrites (or significantly expands) a document
  • Fixes a typo, bug, or other error

@Jxck Jxck requested a review from a team as a code owner May 13, 2022 17:59
@Jxck Jxck requested review from teoli2003 and removed request for a team May 13, 2022 17:59
@github-actions github-actions bot added the Content:HTTP HTTP docs label May 13, 2022
@Jxck
Copy link
Contributor Author

Jxck commented May 13, 2022

hope @sideshowbarker fixing my writing and @mnot reviewing technical details.

@github-actions
Copy link
Contributor

github-actions bot commented May 13, 2022

Preview URLs

Flaws

None! 🎉

External URLs

URL: /en-US/docs/Web/HTTP/Caching
Title: HTTP caching
on GitHub

(this comment was updated 2022-05-16 09:35:08.208811)

@teoli2003 teoli2003 requested review from sideshowbarker and removed request for teoli2003 May 13, 2022 18:09
@sideshowbarker sideshowbarker self-assigned this May 14, 2022
files/en-us/web/http/caching/index.md Outdated Show resolved Hide resolved
files/en-us/web/http/caching/index.md Outdated Show resolved Hide resolved
files/en-us/web/http/caching/index.md Outdated Show resolved Hide resolved
files/en-us/web/http/caching/index.md Outdated Show resolved Hide resolved
files/en-us/web/http/caching/index.md Outdated Show resolved Hide resolved
files/en-us/web/http/caching/index.md Outdated Show resolved Hide resolved
files/en-us/web/http/caching/index.md Outdated Show resolved Hide resolved
files/en-us/web/http/caching/index.md Outdated Show resolved Hide resolved
files/en-us/web/http/caching/index.md Outdated Show resolved Hide resolved
files/en-us/web/http/caching/index.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@sideshowbarker sideshowbarker left a comment

Choose a reason for hiding this comment

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

Amazingly great work as usual

@sideshowbarker sideshowbarker changed the title update caching based on RFC9111 Update caching based on RFC9111 May 16, 2022
@sideshowbarker sideshowbarker merged commit 2103e40 into mdn:main May 16, 2022
@Jxck
Copy link
Contributor Author

Jxck commented May 16, 2022

@sideshowbarker thanks for quic response. But I think it's better reviewed by @mnot too before merging since it's changes old contents a lot and affects caching recommendation widely.

@sideshowbarker
Copy link
Collaborator

@sideshowbarker thanks for quic response. But I think it's better reviewed by @mnot too before merging since it's changes old contents a lot and affects caching recommendation widely.

There's no guarantee that Mark is necessarily going to have time to review it. And anyway, it can still be reviewed post-commit. Any changes can be made in a follow-up PR.

@Jxck Jxck mentioned this pull request May 21, 2022
3 tasks
hamishwillee pushed a commit to hamishwillee/content that referenced this pull request May 23, 2022
Co-authored-by: Michael[tm] Smith <mike@w3.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:HTTP HTTP docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants