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

how to add new user? #527

Closed
zengbotang opened this issue Sep 26, 2016 · 12 comments
Closed

how to add new user? #527

zengbotang opened this issue Sep 26, 2016 · 12 comments
Assignees
Milestone

Comments

@zengbotang
Copy link
Contributor

we are now use the default user:admin@admin.com,
how to add other user?
I can not find any entry in the crane web page.
thanks

@cmingxu
Copy link
Contributor

cmingxu commented Sep 26, 2016

@zengbotang account/user feature currently in active development right now, if you want use your own UI the API are ready for you now to create / list / remove a account.

@zengbotang
Copy link
Contributor Author

Thanks @cmingxu Could you tell me where to find the api doc or the path of these api in the src?

@vitan
Copy link
Contributor

vitan commented Sep 27, 2016

@xiaods
Copy link
Contributor

xiaods commented Sep 27, 2016

this request can be as a milestone.

@vitan vitan added this to the v1.0.6 milestone Sep 27, 2016
@xiaods xiaods self-assigned this Oct 9, 2016
@xiaods
Copy link
Contributor

xiaods commented Oct 10, 2016

the account resource design is not ideal for UCP purpose. login and logout is useful for Auth. so i don't think current api design is good fit and elegant way to handle user activity api.

@xiaods
Copy link
Contributor

xiaods commented Oct 11, 2016

the account api design is bad ideal

@xiaods
Copy link
Contributor

xiaods commented Oct 11, 2016

go through with api documents, in account plugin, the design is
firstly, create group, then create user account. does it make sense? no idea.

@xiaods
Copy link
Contributor

xiaods commented Oct 11, 2016

$ curl -XPOST http://crane.shurenyun.com/account/v1/login -d '{
→ "Email": “admin@admin.com”,
→ "Password":“adminadmin”
}'→ }'
{"code":12006,"data":{},"message":"invalid character 'â' looking for beginning of value","source":"crane"}

@xiaods
Copy link
Contributor

xiaods commented Oct 11, 2016

curl -XGET -H 'Authorization: 08c2b88280865955db7b9c749166bfb3' http://crane.shurenyun.com/account/v1/accounts

@xiaods
Copy link
Contributor

xiaods commented Oct 11, 2016

recap:

  1. API docs need update
  2. from design and project beginning, we prefer don't need add UI design and logic for CRUD on resource, such as plugin: account.
  3. enhance testing and testing againg.

@xiaods xiaods closed this as completed Oct 11, 2016
@vitan
Copy link
Contributor

vitan commented Oct 25, 2016

why close this issue? rename it to update api doc

@vitan vitan reopened this Oct 25, 2016
@vitan
Copy link
Contributor

vitan commented Oct 25, 2016

track the issue at #558

@vitan vitan closed this as completed Oct 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

4 participants