[webmaker-loginapi] Add "deleteUser()", "addUser()" and "editUser()" methods

RESOLVED INVALID

Status

Webmaker
Login
RESOLVED INVALID
5 years ago
5 years ago

People

(Reporter: sedge, Assigned: sedge)

Tracking

Details

(Whiteboard: u=dev c=login p=1 )

(Assignee)

Description

5 years ago
The module needs "delete" functionality.
(Assignee)

Updated

5 years ago
Summary: [webmaker-loginapi] Add "deleteUser()" method → [webmaker-loginapi] Add "deleteUser()", "addUser()" and "editUser()" methods
(Assignee)

Comment 1

5 years ago
...and the rest of the CRUD suite of operations.

Comment 2

5 years ago
This doesn't need to be done for June 15th I think; there's nothing we can edit after an account is created, right?
(Assignee)

Updated

5 years ago
Whiteboard: u=dev c=login p=1 s=2013w23 → u=dev c=login p=1
(Assignee)

Comment 3

5 years ago
This bug exists under the assumption that a server OTHER than the login server will handle serving a page that webmakers can use to modify their own accounts.  

Currently, we have the "/account" page served from Login. If this evolves into a full self-management console for user accounts then this bug is invalid.

Thoughts?
Flags: needinfo?(kate)
Flags: needinfo?(jon)
Ideally we'd like to have everything in one place, which will probably be webmaker.org -- so moving the /account page to webmaker.org after this bug is fixed is something we'd like to do, yes.
Flags: needinfo?(kate)
(Assignee)

Comment 5

5 years ago
Okay, so then the next question is should the module attach a route for the other crud operations too? Like how it attaches `GET /user/:userid`, which proxies to `GET login.wm.o/user/:id`

Comment 6

5 years ago
I don't think we should worry about implementing any of this functionality for other services until we can actually edit user accounts safely. You can't change your email or your username; these are things that users would probably like to change.
Flags: needinfo?(jon)
(Assignee)

Updated

5 years ago
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.