/juju

jujuclient: add AccountStore #4379

Merged
merged 2 commits into from Feb 11, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@axw @jujubot
@axw
Member

axw commented Feb 11, 2016

Basically just a find/replace of Model with
Account.

Accounts are currently keyed by user name,
but we also store the user name in the account
details in case we want to separate the name
of the account from the user.

(Review request: http://reviews.vapour.ws/r/3822/)

axw added some commits Feb 11, 2016

@axw
jujuclient: add AccountStore 
Basically just a find/replace of Model with
Account.

Accounts are currently keyed by user name,
but we also store the user name in the account
details in case we want to separate the name
of the account from the user.
2ac6a6f
@axw
Add comments
3e40b5a
@axw
Member

axw commented Feb 11, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Feb 11, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

jujubot added a commit that referenced this pull request Feb 11, 2016

@jujubot
Merge pull request #4379 from axw/cloud-credentials-accountstore 
jujuclient: add AccountStore

Basically just a find/replace of Model with
Account.

Accounts are currently keyed by user name,
but we also store the user name in the account
details in case we want to separate the name
of the account from the user.

(Review request: http://reviews.vapour.ws/r/3822/)
d2677fe

@jujubot jujubot merged commit d2677fe into juju:cloud-credentials Feb 11, 2016

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