timbru31 commented on a change in pull request #1043: Document possible IndexedDB eviction with MDN ref URL: https://github.com/apache/cordova-docs/pull/1043#discussion_r361942167
########## File path: www/docs/en/dev/cordova/storage/storage.md ########## @@ -270,7 +270,7 @@ For more information, see: - Not supported on iOS. - Complex API with nested callbacks. -- Limited total amount of storage (typically around 5MB). +- Limited total amount of storage (typically around 5MB) and possible eviction [as described on MDN][MDNIndexedDBLimitsAndEviction]. Review comment: 5MB is totally wrong. We should update this value, too. IMO we should remove the braces statement and just link to MDN. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
