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

Delete RTCStatsType enum #17336

Merged
merged 2 commits into from
Jun 20, 2022
Merged

Delete RTCStatsType enum #17336

merged 2 commits into from
Jun 20, 2022

Conversation

teoli2003
Copy link
Contributor

This removes the RTCStatsType that is the last enum preventing us to finish the API page-type project (there are still a few typedefs to remove).

This is part of #16255

@teoli2003 teoli2003 requested review from a team as code owners June 16, 2022 06:50
@teoli2003 teoli2003 requested review from a team, sideshowbarker and fiji-flo and removed request for a team June 16, 2022 06:50
@github-actions github-actions bot added Content:Other Any docs not covered by another "Content:" label Content:WebAPI Web API docs labels Jun 16, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Jun 16, 2022

Preview URLs

Flaws

Note! 2 documents with no flaws that don't need to be listed. 🎉

URL: /en-US/docs/Web/API/RTCStats
Title: RTCStats
on GitHub
Flaw count: 4

  • macros:
    • /en-US/docs/Web/API/RTCReceivedRtpStreamStats does not exist
    • /en-US/docs/Web/API/RTCReceivedRtpStreamStats does not exist
    • /en-US/docs/Web/API/RTCSentRtpStreamStats does not exist
  • bad_bcd_queries:
    • No BCD data for query: api.RTCStats

URL: /en-US/docs/Web/API/RTCStats/type
Title: RTCStats.type
on GitHub
Flaw count: 18

  • macros:
    • /en-US/docs/Web/API/RTCCertificateStats does not exist
    • /en-US/docs/Web/API/RTCCodecStats does not exist
    • /en-US/docs/Web/API/RTCContributingSourceStats does not exist
    • /en-US/docs/Web/API/RTCDataChannelStats does not exist
    • /en-US/docs/Web/API/RTCPeerConnectionStats does not exist
    • and 12 more flaws omitted
  • bad_bcd_queries:
    • No BCD data for query: api.RTCStats.type

URL: /en-US/docs/Web/API/RTCRemoteOutboundRtpStreamStats/localId
Title: RTCRemoteOutboundRtpStreamStats.localId
on GitHub
Flaw count: 2

  • macros:
    • /en-US/docs/Web/API/RTCSentRtpStreamStats/packetsSent does not exist
    • /en-US/docs/Web/API/RTCSentRtpStreamStats/bytesSent does not exist

URL: /en-US/docs/Web/API/WebRTC_Statistics_API
Title: WebRTC Statistics API
on GitHub
Flaw count: 35

  • macros:
    • /en-US/docs/Web/API/RTCCertificateStats does not exist
    • /en-US/docs/Web/API/RTCCodecStats does not exist
    • /en-US/docs/Web/API/RTCRtpContributingSourceStats does not exist
    • /en-US/docs/Web/API/RTCDataChannelStats does not exist
    • /en-US/docs/Web/API/RTCIceServerStats does not exist
    • and 30 more flaws omitted

External URLs

URL: /en-US/docs/Web/API/RTCStats
Title: RTCStats
on GitHub

No new external URLs


URL: /en-US/docs/Web/API/RTCStats/type
Title: RTCStats.type
on GitHub

No new external URLs


URL: /en-US/docs/Web/API/RTCRemoteOutboundRtpStreamStats/localId
Title: RTCRemoteOutboundRtpStreamStats.localId
on GitHub

No new external URLs


URL: /en-US/docs/Web/API/WebRTC_Statistics_API
Title: WebRTC Statistics API
on GitHub

No new external URLs


URL: /en-US/docs/Web/API/RTCStatsReport
Title: RTCStatsReport
on GitHub

No new external URLs


URL: /en-US/docs/Mozilla/Firefox/Releases/79
Title: Firefox 79 for developers
on GitHub

No new external URLs

(this comment was updated 2022-06-20 19:50:03.167198)

Copy link
Collaborator

@wbamberg wbamberg left a comment

Choose a reason for hiding this comment

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

👍 looks good, thank you @teoli2003 !

@wbamberg wbamberg merged commit d557a2d into mdn:main Jun 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:Other Any docs not covered by another "Content:" label Content:WebAPI Web API docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants