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

feat(HTTP): Refresh 3XX response pages #35353

Merged
merged 18 commits into from
Sep 2, 2024
Merged

Conversation

bsmth
Copy link
Member

@bsmth bsmth commented Aug 7, 2024

Description

This PR refreshes the 3XX HTTP status response code pages.

Motivation

Keeping content up-to-date and useful.

additions:

  • Examples in HTTP format
  • See also links to HTTP status codes page
  • Unify formatting with current conventions (e.g., ## Example to ## Examples with subsections)

Related issues and pull requests:

@bsmth bsmth requested a review from a team as a code owner August 7, 2024 09:16
@bsmth bsmth requested review from hamishwillee and removed request for a team August 7, 2024 09:16
@github-actions github-actions bot added Content:HTTP HTTP docs size/m [PR only] 51-500 LoC changed labels Aug 7, 2024
Copy link
Contributor

github-actions bot commented Aug 7, 2024

Preview URLs (7 pages)
External URLs (6)

URL: /en-US/docs/Web/HTTP/Status/304
Title: 304 Not Modified


URL: /en-US/docs/Web/HTTP/Status/303
Title: 303 See Other


URL: /en-US/docs/Web/HTTP/Status/300
Title: 300 Multiple Choices


URL: /en-US/docs/Web/HTTP/Status/308
Title: 308 Permanent Redirect

(comment last updated: 2024-08-30 10:55:18)

@bsmth
Copy link
Member Author

bsmth commented Aug 28, 2024

Thanks Hamish! Incorporated your feedback if you want to have another look 👀

@bsmth bsmth requested a review from hamishwillee August 28, 2024 11:19
@hamishwillee
Copy link
Collaborator

Nearly there :-)

Copy link
Collaborator

@hamishwillee hamishwillee left a comment

Choose a reason for hiding this comment

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

Thanks @bsmth - so much better than it was previously.

@hamishwillee hamishwillee merged commit fcb01c3 into mdn:main Sep 2, 2024
8 checks passed
@bsmth bsmth deleted the http-3xx-statuses branch September 2, 2024 08:22
@bsmth
Copy link
Member Author

bsmth commented Sep 2, 2024

Thanks, Hamish

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:HTTP HTTP docs size/m [PR only] 51-500 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants