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
fix #308 #411
fix #308 #411
Conversation
@@ -4,7 +4,9 @@ | |||
Route::resource('tickets', 'TicketsController', ['except' => 'destroy']); | |||
Route::post('tickets/{ticket}/comments', 'CommentsController@store'); | |||
Route::post('tickets/{ticket}/assign', 'TicketAssignController@store'); | |||
Route::post('users/create', 'UsersController@store'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
actually the create
is not required, a simple post to users will create them
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well an endpoint to create and one to list them, I can change the endpoint name in case
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, POST users
to create one, GET users
list them
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GET users/{user_id}
to the full info of one
$users=new User; | ||
$users->name=$request->get('name'); | ||
$users->email=$request->get('email'); | ||
$users->password=Hash::make($request->get('password')); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we usually use the bcrypt method bcrypt($request('password'))
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I saw that in the handesk code is used hash
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh, ok then ;)
So I don't need to add any tests for that? |
A test would be great!! |
This add 2 new endpoints:
Require a bit of testing because I cannot find how to use the endpoints in handesk