GitToTheHub commented on code in PR #1455:
URL: https://github.com/apache/cordova-docs/pull/1455#discussion_r2662116309


##########
www/docs/en/dev/cordova/storage/storage.md:
##########
@@ -187,6 +188,66 @@ For more information, see:
 - Limited total amount of storage and possible eviction [as described on 
MDN][MDNIndexedDBLimitsAndEviction].
 - On iOS systems, WebKit may delete web-related data storage when the device 
is under storage pressure or exceeds defined storage quotas. This eviction is 
triggered by WebKit itself, based on its internal quota management policies. 
For more information, see [WebKit – Updates to Storage 
Policy][WebKitUpdatesToStoragePolicy].
 
+## Cache Storage
+
+### Overview
+
+The Cache Storage API is a web-standard storage mechanism primarily used for
+storing network request and response pairs. It is commonly used to support
+resource caching and offline-read scenarios, and is often associated with
+Service Worker–based caching strategies.
+
+### Availability in Cordova
+
+Cache Storage is exposed via the standard `window.caches` interface when 
supported
+by the underlying WebView implementation. Availability and behavior depend 
entirely
+on the platform WebView, and the API may not be available on all Cordova 
platforms
+or in all execution contexts.
+
+### Platform behavior and caveats
+
+Behavior of Cache Storage in Cordova applications can differ from typical 
desktop
+browser environments:
+
+- **Android (Chromium WebView):** Behavior is generally similar to Chrome, but 
may
+  vary depending on the WebView version shipped with the device.

Review Comment:
   Could this be explained further what `but may vary depending on the WebView 
version shipped with the device.`? An example maybe?



##########
www/docs/en/dev/cordova/storage/storage.md:
##########
@@ -187,6 +188,66 @@ For more information, see:
 - Limited total amount of storage and possible eviction [as described on 
MDN][MDNIndexedDBLimitsAndEviction].
 - On iOS systems, WebKit may delete web-related data storage when the device 
is under storage pressure or exceeds defined storage quotas. This eviction is 
triggered by WebKit itself, based on its internal quota management policies. 
For more information, see [WebKit – Updates to Storage 
Policy][WebKitUpdatesToStoragePolicy].
 
+## Cache Storage
+
+### Overview
+
+The Cache Storage API is a web-standard storage mechanism primarily used for
+storing network request and response pairs. It is commonly used to support
+resource caching and offline-read scenarios, and is often associated with
+Service Worker–based caching strategies.
+
+### Availability in Cordova
+
+Cache Storage is exposed via the standard `window.caches` interface when 
supported
+by the underlying WebView implementation. Availability and behavior depend 
entirely
+on the platform WebView, and the API may not be available on all Cordova 
platforms

Review Comment:
   `and the API may not be available on all Cordova platforms`: This sounds 
also, like this has something to do exclusively with Cordova but it's not. 
Maybe it could be `and the API may not be available on all Cordova supported 
platforms`



##########
www/docs/en/dev/cordova/storage/storage.md:
##########
@@ -187,6 +188,66 @@ For more information, see:
 - Limited total amount of storage and possible eviction [as described on 
MDN][MDNIndexedDBLimitsAndEviction].
 - On iOS systems, WebKit may delete web-related data storage when the device 
is under storage pressure or exceeds defined storage quotas. This eviction is 
triggered by WebKit itself, based on its internal quota management policies. 
For more information, see [WebKit – Updates to Storage 
Policy][WebKitUpdatesToStoragePolicy].
 
+## Cache Storage
+
+### Overview
+
+The Cache Storage API is a web-standard storage mechanism primarily used for
+storing network request and response pairs. It is commonly used to support
+resource caching and offline-read scenarios, and is often associated with
+Service Worker–based caching strategies.
+
+### Availability in Cordova

Review Comment:
   It sounds like this is something related Cordova, but it's not. It should be 
just `Availability`



##########
www/docs/en/dev/cordova/storage/storage.md:
##########
@@ -187,6 +188,66 @@ For more information, see:
 - Limited total amount of storage and possible eviction [as described on 
MDN][MDNIndexedDBLimitsAndEviction].
 - On iOS systems, WebKit may delete web-related data storage when the device 
is under storage pressure or exceeds defined storage quotas. This eviction is 
triggered by WebKit itself, based on its internal quota management policies. 
For more information, see [WebKit – Updates to Storage 
Policy][WebKitUpdatesToStoragePolicy].
 
+## Cache Storage
+
+### Overview
+
+The Cache Storage API is a web-standard storage mechanism primarily used for
+storing network request and response pairs. It is commonly used to support
+resource caching and offline-read scenarios, and is often associated with
+Service Worker–based caching strategies.
+
+### Availability in Cordova
+
+Cache Storage is exposed via the standard `window.caches` interface when 
supported
+by the underlying WebView implementation. Availability and behavior depend 
entirely
+on the platform WebView, and the API may not be available on all Cordova 
platforms
+or in all execution contexts.
+
+### Platform behavior and caveats
+
+Behavior of Cache Storage in Cordova applications can differ from typical 
desktop

Review Comment:
   `Behavior of Cache Storage in Cordova applications can differ from typical 
desktop` could be `Behavior of Cache Storage in mobile applications can differ 
from typical desktop`



##########
www/docs/en/dev/cordova/storage/storage.md:
##########
@@ -187,6 +188,66 @@ For more information, see:
 - Limited total amount of storage and possible eviction [as described on 
MDN][MDNIndexedDBLimitsAndEviction].
 - On iOS systems, WebKit may delete web-related data storage when the device 
is under storage pressure or exceeds defined storage quotas. This eviction is 
triggered by WebKit itself, based on its internal quota management policies. 
For more information, see [WebKit – Updates to Storage 
Policy][WebKitUpdatesToStoragePolicy].
 
+## Cache Storage
+
+### Overview
+
+The Cache Storage API is a web-standard storage mechanism primarily used for
+storing network request and response pairs. It is commonly used to support
+resource caching and offline-read scenarios, and is often associated with
+Service Worker–based caching strategies.
+
+### Availability in Cordova
+
+Cache Storage is exposed via the standard `window.caches` interface when 
supported
+by the underlying WebView implementation. Availability and behavior depend 
entirely
+on the platform WebView, and the API may not be available on all Cordova 
platforms
+or in all execution contexts.
+
+### Platform behavior and caveats
+
+Behavior of Cache Storage in Cordova applications can differ from typical 
desktop
+browser environments:
+
+- **Android (Chromium WebView):** Behavior is generally similar to Chrome, but 
may
+  vary depending on the WebView version shipped with the device.
+- **iOS (WKWebView):** Cache Storage persistence is more restrictive, and 
cached
+  data may be evicted more aggressively by the system.
+- **Origin and scheme sensitivity:** Access to cached data is origin-dependent.
+  Changes to the application origin, scheme, or WebView implementation may 
prevent
+  access to previously stored cache entries.
+
+### Persistence and eviction
+
+Cache Storage should not be considered durable storage. Cached data may be 
removed
+by the operating system or WebView implementation under storage pressure, 
during
+application updates, or when the application is reinstalled.
+
+As with other web storage mechanisms, applications should assume that cached 
data
+can be evicted at any time and should be able to recover gracefully.
+
+### When to use (and when not to)
+
+Cache Storage is appropriate for:
+
+- Caching network resources such as API responses or static assets
+- Reducing network usage
+- Supporting offline-read or offline-first scenarios
+
+Cache Storage is **not** appropriate for:
+
+- Storing user-generated or business-critical data
+- Data that requires guaranteed persistence across app restarts or updates
+
+For structured or durable data storage, consider using IndexedDB instead.
+
+
+### References
+
+- [MDN: Cache Storage 
API](https://developer.mozilla.org/en-US/docs/Web/API/CacheStorage)

Review Comment:
   I would write
   - [MDN: CacheStorage 
API](https://developer.mozilla.org/en-US/docs/Web/API/CacheStorage) - Handles 
all caches



##########
www/docs/en/dev/cordova/storage/storage.md:
##########
@@ -187,6 +188,66 @@ For more information, see:
 - Limited total amount of storage and possible eviction [as described on 
MDN][MDNIndexedDBLimitsAndEviction].
 - On iOS systems, WebKit may delete web-related data storage when the device 
is under storage pressure or exceeds defined storage quotas. This eviction is 
triggered by WebKit itself, based on its internal quota management policies. 
For more information, see [WebKit – Updates to Storage 
Policy][WebKitUpdatesToStoragePolicy].
 
+## Cache Storage
+
+### Overview
+
+The Cache Storage API is a web-standard storage mechanism primarily used for
+storing network request and response pairs. It is commonly used to support
+resource caching and offline-read scenarios, and is often associated with
+Service Worker–based caching strategies.
+
+### Availability in Cordova
+
+Cache Storage is exposed via the standard `window.caches` interface when 
supported
+by the underlying WebView implementation. Availability and behavior depend 
entirely
+on the platform WebView, and the API may not be available on all Cordova 
platforms
+or in all execution contexts.
+
+### Platform behavior and caveats
+
+Behavior of Cache Storage in Cordova applications can differ from typical 
desktop
+browser environments:
+
+- **Android (Chromium WebView):** Behavior is generally similar to Chrome, but 
may
+  vary depending on the WebView version shipped with the device.
+- **iOS (WKWebView):** Cache Storage persistence is more restrictive, and 
cached
+  data may be evicted more aggressively by the system.

Review Comment:
   What does this mean `is more restrictive` and `cached data may be evicted 
more aggressively by the system.`. An example would be good to explain, what 
this means.



##########
www/docs/en/dev/cordova/storage/storage.md:
##########
@@ -187,6 +188,66 @@ For more information, see:
 - Limited total amount of storage and possible eviction [as described on 
MDN][MDNIndexedDBLimitsAndEviction].
 - On iOS systems, WebKit may delete web-related data storage when the device 
is under storage pressure or exceeds defined storage quotas. This eviction is 
triggered by WebKit itself, based on its internal quota management policies. 
For more information, see [WebKit – Updates to Storage 
Policy][WebKitUpdatesToStoragePolicy].
 
+## Cache Storage
+
+### Overview
+
+The Cache Storage API is a web-standard storage mechanism primarily used for
+storing network request and response pairs. It is commonly used to support
+resource caching and offline-read scenarios, and is often associated with
+Service Worker–based caching strategies.
+
+### Availability in Cordova
+
+Cache Storage is exposed via the standard `window.caches` interface when 
supported
+by the underlying WebView implementation. Availability and behavior depend 
entirely
+on the platform WebView, and the API may not be available on all Cordova 
platforms
+or in all execution contexts.
+
+### Platform behavior and caveats
+
+Behavior of Cache Storage in Cordova applications can differ from typical 
desktop
+browser environments:
+
+- **Android (Chromium WebView):** Behavior is generally similar to Chrome, but 
may
+  vary depending on the WebView version shipped with the device.
+- **iOS (WKWebView):** Cache Storage persistence is more restrictive, and 
cached
+  data may be evicted more aggressively by the system.
+- **Origin and scheme sensitivity:** Access to cached data is origin-dependent.
+  Changes to the application origin, scheme, or WebView implementation may 
prevent
+  access to previously stored cache entries.

Review Comment:
   The 
[CacheStorage](https://developer.mozilla.org/en-US/docs/Web/API/CacheStorage) 
notes also:
   
   > This feature is available only in [secure 
contexts](https://developer.mozilla.org/en-US/docs/Web/Security/Defenses/Secure_Contexts)
 (HTTPS)
   
   This should be added.
   
   Also it could be added what the minimum Platform is:
   - Android: Android WebView version 65, Android 5.0
   - iOS: 11.3
   
   (Source: 
https://developer.mozilla.org/en-US/docs/Web/API/CacheStorage#browser_compatibility)



##########
www/docs/en/dev/cordova/storage/storage.md:
##########
@@ -187,6 +188,66 @@ For more information, see:
 - Limited total amount of storage and possible eviction [as described on 
MDN][MDNIndexedDBLimitsAndEviction].
 - On iOS systems, WebKit may delete web-related data storage when the device 
is under storage pressure or exceeds defined storage quotas. This eviction is 
triggered by WebKit itself, based on its internal quota management policies. 
For more information, see [WebKit – Updates to Storage 
Policy][WebKitUpdatesToStoragePolicy].
 
+## Cache Storage
+
+### Overview
+
+The Cache Storage API is a web-standard storage mechanism primarily used for
+storing network request and response pairs. It is commonly used to support
+resource caching and offline-read scenarios, and is often associated with
+Service Worker–based caching strategies.
+
+### Availability in Cordova
+
+Cache Storage is exposed via the standard `window.caches` interface when 
supported
+by the underlying WebView implementation. Availability and behavior depend 
entirely
+on the platform WebView, and the API may not be available on all Cordova 
platforms
+or in all execution contexts.
+
+### Platform behavior and caveats
+
+Behavior of Cache Storage in Cordova applications can differ from typical 
desktop
+browser environments:
+
+- **Android (Chromium WebView):** Behavior is generally similar to Chrome, but 
may
+  vary depending on the WebView version shipped with the device.
+- **iOS (WKWebView):** Cache Storage persistence is more restrictive, and 
cached
+  data may be evicted more aggressively by the system.
+- **Origin and scheme sensitivity:** Access to cached data is origin-dependent.
+  Changes to the application origin, scheme, or WebView implementation may 
prevent
+  access to previously stored cache entries.
+
+### Persistence and eviction
+
+Cache Storage should not be considered durable storage. Cached data may be 
removed
+by the operating system or WebView implementation under storage pressure, 
during
+application updates, or when the application is reinstalled.
+
+As with other web storage mechanisms, applications should assume that cached 
data
+can be evicted at any time and should be able to recover gracefully.
+
+### When to use (and when not to)
+
+Cache Storage is appropriate for:
+
+- Caching network resources such as API responses or static assets
+- Reducing network usage
+- Supporting offline-read or offline-first scenarios
+
+Cache Storage is **not** appropriate for:
+
+- Storing user-generated or business-critical data
+- Data that requires guaranteed persistence across app restarts or updates
+
+For structured or durable data storage, consider using IndexedDB instead.
+
+
+### References
+
+- [MDN: Cache Storage 
API](https://developer.mozilla.org/en-US/docs/Web/API/CacheStorage)
+- [MDN: Using the Cache 
API](https://developer.mozilla.org/en-US/docs/Web/API/Cache)

Review Comment:
   Here:
   - [MDN: Cache API](https://developer.mozilla.org/en-US/docs/Web/API/Cache) - 
Work with a single cache



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to