content icon indicating copy to clipboard operation
content copied to clipboard

Delete WebAssembly/Caching_modules

Open dawei-wang opened this issue 3 years ago • 4 comments

Fixes mdn/content#14520

Description

Motivation

Additional details

Related issues and pull requests

dawei-wang avatar Nov 07 '22 22:11 dawei-wang

Preview URLs

Flaws (4)

URL: /en-US/docs/WebAssembly/JavaScript_interface/instantiate Title: WebAssembly.instantiate() Flaw count: 4

  • broken_links:
    • Can't resolve /en-US/docs/WebAssembly/Caching_modules
    • Link points to the page it's already on
    • Link points to the page it's already on
  • bad_bcd_links:
    • no explanation!

(this comment was updated 2022-11-11 03:21:41.355497)

github-actions[bot] avatar Nov 07 '22 22:11 github-actions[bot]

Given @https://github.com/mdn/content/issues/14520#issuecomment-1306404876, requested review from @hamishwillee

sideshowbarker avatar Nov 08 '22 03:11 sideshowbarker

@dawei-wang Thank you for this. I just put a comment in the corresponding issue that I "think" that it is good to remove this material, but that I think that it is very likely that developers will still want to cache WASM modules.

So what I think we need ideally is to replace this content showing how to use some other cache than indexDB. Do you think that this is something you could do? If would be better to do now because the existing doc is here so you could copy the current structue.

I appreciate that's a much bigger job than deleting this!

If you don't want to do that then let me know and I'll review this on the assumption that we are deleting this material but keeping a red-link to the old doc.

hamishwillee avatar Nov 08 '22 03:11 hamishwillee

@hamishwillee I'm not familiar with caching mechanisms enough to document a new one. Let's keep a red link to the old doc.

dawei-wang avatar Nov 09 '22 19:11 dawei-wang

OK, Merging. Thanks @dawei-wang

hamishwillee avatar Nov 15 '22 00:11 hamishwillee