Skip to content
This repository was archived by the owner on Dec 12, 2018. It is now read-only.

Conversation

peebles
Copy link

@peebles peebles commented Sep 7, 2015

  1. getCacheByHref() still incorrect, missing this case:
    https://api.stormpath.com/v1/applications/6fr66lFwpF8NTmFH0kITot/groups
  2. Add recursive CacheHandler::get()
  3. Recursive CacheHandler::put() was not storing the toplevel resource in
    some cases.
  4. If process.env.NODE_DEBUG_SP=1, then print some debug statements when Stormpath
    fetches from cloud uri verses from cache. Also stores to cache are displayed.

    https://api.stormpath.com/v1/applications/6fr66lFwpF8NTmFH0kITot/groups

2.  Add recursive CacheHandler::get()

3.  Recursive CacheHandler::put() was not storing the toplevel resource in
    some cases.

4.  If process.env.NODE_DEBUG_SP=1, then print some debug statements when Stormpath
    fetches from cloud uri verses from cache.  Also stores to cache are displayed.
@peebles
Copy link
Author

peebles commented Sep 8, 2015

I see its failing lint checks, although it seems like the checks are pretty strict. If there was a lint "script" in package.json, I could run it and fix the lint errors, but as it I wouldn't know how you are running lint. Let me know if you want me to do anything to help.

I noticed the npm module from doing a "npm install storm" is pretty far behind master...

@peebles
Copy link
Author

peebles commented Sep 8, 2015

I found that running "npm run test" ran lint. I fixed the lint errors.

@robertjd
Copy link
Member

robertjd commented Nov 6, 2015

Hi @peebles, thanks for this pull request. We've been busy with other features, but we will definitely review this as soon as possible.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants