store sandbox using yui-sandbox #636

Merged
merged 5 commits into from Oct 16, 2012

Projects

None yet

3 participants

@caridy
Collaborator
caridy commented Oct 16, 2012
  • isolation between store instance and the mojito Y instace
  • remove experimental dataprocess
@mojit0
Contributor
mojit0 commented Oct 16, 2012

+1

@caridy
Collaborator
caridy commented Oct 16, 2012

let's wait until after pr2 patch happens to merge this.

@caridy caridy using basic JSON stringify/parse for caching strings rather than obje…
…cts to speed up the response now that we are not using DataProcess anymore which impacts perf.
c1115af
@caridy
Collaborator
caridy commented Oct 16, 2012

Travis is failing due to a timeout rather than a code problem I guess: "Executing your script (./travis/script.sh) took longer than 30 minutes and was terminated."

@drewfish
Member

+1. I think that store.server.js should probably be using yui-sandbox instead, for when it uses Y.Loader.

@caridy
Collaborator
caridy commented Oct 16, 2012

Yes, part of the next refactor. Trello card #88 :)

@caridy caridy merged commit 8b89ad9 into yahoo:develop-perf Oct 16, 2012

1 check failed

default The Travis build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment