Fix issue 33356: Document integrity properly

[wbamberg]

Fixes #33356.

[github-actions]

Preview URLs

• /en-US/docs/Web/API/Request/integrity

(comment last updated: 2024-06-26 08:23:19)

[Josh-Cena]

I'm not sure if this page is the right place to document it as it's a simple reflector of the internal state of the request. It should probably be on the Request() and/or fetch() page (before we merge them, that is), and a brief mention can be put here.

[wbamberg]

I'm not sure if this page is the right place to document it as it's a simple reflector of the internal state of the request. It should probably be on the Request() and/or fetch() page (before we merge them, that is), and a brief mention can be put here.

I did think about this, and the reason I went with here is that we have a whole page to talk about it here, and it's nice to be able to use that space. If we document it in the constructor or fetch() (or even RequestInit) we'd only have one page to cover all the options, and it would get huge. We do something similar in (for example) https://developer.mozilla.org/en-US/docs/Web/API/Request/cache.

Alternative I suppose would be to create subpages for RequestInit, although we usually don't, for dictionaries.

[Josh-Cena]

We can either make a tree for RequestInit, or alternatively just have fetch/integrity, fetch/cache, etc. Not easy to design 😅

I don't have strong opinions against this either. The WebAPI information is really all over the place (especially HTML element stuff).

[wbamberg]

I think the best alternative is inline in RequestInit, and I'm happy to try that when it exists.

[estelle]

Thanks! 🎉

[Josh-Cena]

@wbamberg Do you want to merge this or to move the content to RequestInit instead?

[wbamberg]

I put all the relevant stuff in the other PR: https://pr34437.content.dev.mdn.mozit.cloud/en-US/docs/Web/API/RequestInit#integrity so think we can close this PR without merging.