Skip to content
This repository was archived by the owner on Mar 14, 2024. It is now read-only.

[DevTools 99] Network: WebSocket throttling - #2498

Merged
sofiayem merged 2 commits into
mainfrom
devtools-99-netwrok-throttle
Apr 4, 2022
Merged

sofiayem merged 2 commits into
mainfrom
devtools-99-netwrok-throttle

Conversation

@sofiayem

@sofiayem sofiayem commented Apr 1, 2022 •

Copy link
Copy Markdown
Contributor

Fixes #2520
Fixed #2527

Updated the throttling section.
Added "Throttle WebSocket connections"
@sofiayem sofiayem added the reviewers: devtools Enables code review by the DevTools team. label Apr 1, 2022
@sofiayem
sofiayem requested review from jecfish and rachelandrew April 1, 2022 15:20
@netlify

netlify Bot commented Apr 1, 2022 •

Copy link
Copy Markdown

✅ Deploy Preview for developer-chrome-com ready!

Name Link
🔨 Latest commit c0db3bc
🔍 Latest deploy log https://app.netlify.com/sites/developer-chrome-com/deploys/624adeb6f8125b000806fba9
😎 Deploy Preview https://deploy-preview-2498--developer-chrome-com.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@jecfish jecfish left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Minor nit on the name. Please update as well if any other places use tab instead of panel.

Thanks Sofia!

Comment thread site/en/docs/devtools/network/reference/index.md Outdated
Co-authored-by: Jecelyn Yeen <5917927+jecfish@users.noreply.github.com>
@chrome-devrel-review-bot

Copy link
Copy Markdown
Collaborator

Hello! This is an automated review by our custom reviewbot. It updates automatically when code or GitHub comments in this pull request are created or updated.

Requested changes

If there are any common problems with the content files you created or modified, they will be listed here.

site/en/docs/devtools/network/reference/index.md

  • This file passed all of our automated Markdown audits.

1 similar comment
@chrome-devrel-review-bot

Copy link
Copy Markdown
Collaborator

Hello! This is an automated review by our custom reviewbot. It updates automatically when code or GitHub comments in this pull request are created or updated.

Requested changes

If there are any common problems with the content files you created or modified, they will be listed here.

site/en/docs/devtools/network/reference/index.md

  • This file passed all of our automated Markdown audits.

@sofiayem
sofiayem merged commit e465a8a into main Apr 4, 2022
@sofiayem
sofiayem deleted the devtools-99-netwrok-throttle branch April 4, 2022 12:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

reviewers: devtools Enables code review by the DevTools team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Outdated: Network > Throttling v99: Network > Throttling WebSockets

3 participants