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

old dojox.storage documentation is broken #154

Open
dylans opened this issue Sep 26, 2017 · 0 comments
Open

old dojox.storage documentation is broken #154

dylans opened this issue Sep 26, 2017 · 0 comments

Comments

@dylans
Copy link
Member

dylans commented Sep 26, 2017

@spaceone commented on Mon Sep 18 2017

https://dojotoolkit.org/reference-guide/1.10/dojox/storage.html says to use the deprecated:
dojo.addOnLoad() and the example aren't even working. It seems dojo.addOnLoad is never called.

Could you upgrade the documentation to reflect an modern state?
At least say a word that https://github.com/SitePen/dstore/blob/master/docs/Stores.md could be used instead? Or how to use dojox/storage/LocalStorageProvider as an AMD module?


@dylans commented on Tue Sep 26 2017

Note that this repo is for Dojo 2... so I'm going to move this issue to the dojo/docs repo where we'd be happy to accept a pull request to get this landed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant