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

Application.getDefault...Store() to yield Directory instances #587

Open
wants to merge 3 commits into
base: 1.0.0
Choose a base branch
from

Conversation

the-overengineer
Copy link

Fixed Application.getDefaultAccountStore() and Application.getDefaultGroupStore() to return store (Directory) instances instead of links. Implemented via mixin. Added mixin to Organization, as well. Fixed tests where applicable, removed some from Organization where it gets too complex to test successfully.

Breaking change, hence merge to 1.0.0.

var application;

application.getDefaultAccountStore(function (err, directory) {
  if (err) {
    throw err;
  }

  console.log(directory.name);
});

Fixes #376

@coveralls
Copy link

coveralls commented Dec 20, 2016

Coverage Status

Coverage decreased (-0.3%) to 89.205% when pulling a9648ac on feature/get-store-return-store into c09a89c on 1.0.0.

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

Successfully merging this pull request may close these issues.

3 participants