mothership-ec / cog-mothership-user

Mothership User Cogule
Other
1 stars 1 forks source link

Admin ability to reset User password to be added? #15

Open eleanorshakeshaft opened 11 years ago

lsjroberts commented 11 years ago

Added by ab20c3c08e30db5e77e9af5d0e192185c8ed8113 & 217193e25089f1074b4c1c0976e99c87c3b40e91.

However, the {{ domain }} and {{ resetUrl }} values need to be set in https://github.com/messagedigital/cog-mothership-user/blob/master/src/Message/Mothership/User/Controller/User/DetailsEdit.php#L58-L59.

I'm not sure where we can get the resetUrl since it will change depending on the user's group(s), i.e. admin or customer.

lsjroberts commented 11 years ago

Also waiting on https://github.com/messagedigital/cog/issues/193 before sending to a non-hardcoded email address.

joeholdcroft commented 11 years ago

@lsjroberts whilst I think the functionality to press a button and send the user a forgotten password link email is helpful, it isn't something we need in v1.0.0-beta1 so please remove the button for now and raise an issue for it to be re-instated later when it's working correctly.

However, we **do* need the functionality (and I think this is what @eleanorshakeshaft meant by the issue title) for the Admin to just re-set the password in the control panel.

Again, this is something we may want to restrict to only certain users (see issue #16).

lsjroberts commented 11 years ago

@joeholdcroft Removed checkbox with 1c9363c

eleanorshakeshaft commented 9 years ago

There's unused code relating to this here: https://github.com/messagedigital/cog-mothership-user/blob/develop/src/Controller/User/DetailsEdit.php#L48