Peertube: option for to admin verify user email

Created on 31 Aug 2018  路  5Comments  路  Source: Chocobozzz/PeerTube

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.

Type

Most helpful comment

Thanks. Glad I asked - that's definitely the simplest. I'll try to get this done by Wednesday latest.

All 5 comments

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.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

Nutomic picture Nutomic  路  3Comments

XenonFiber picture XenonFiber  路  3Comments

Jorropo picture Jorropo  路  3Comments

milleniumbug picture milleniumbug  路  3Comments

Angedestenebres picture Angedestenebres  路  3Comments