As discussed here #977 - this would allow the admin to verify a user's email to the console. This is mainly to address the edge case where a user registers when email verification is required and there are technical issues around emails that can't quickly be resolved.
As I said in https://github.com/Chocobozzz/PeerTube/pull/977#issuecomment-416868853 I think it would be better to have checkbox in the admin user form.
For sure - just a checkbox. Should be simple. You can assign me
@Chocobozzz - apologies for the lag on resolving this issue but should have this done soon. Started working on it today and realized there are one of two ways to do this on the back-end. Looking for your preference before going too far down either path.
Currently the :userId/verify-email route validates for a verificationString then checks against redis. So either:
1) Create a completely new route for verification by admin - e.g. /verify-emails with array of user ids in body.
2) Use same route, but check for either verificationString or verifiedByAdmin = true. If verifiedByAdmin = true then don't check redis. verifiedByAdmin = true would be added to request body in user-list.component
Or add emailVerified to the UserUpdate model and use the update endpoint? https://github.com/Chocobozzz/PeerTube/blob/develop/server/controllers/api/users/index.ts#L258
Thanks. Glad I asked - that's definitely the simplest. I'll try to get this done by Wednesday latest.
Most helpful comment
Thanks. Glad I asked - that's definitely the simplest. I'll try to get this done by Wednesday latest.