search

MultiEmail / backend

Project to manage multiple emails at once with lots of customization. You can send and receive emails. Desktop notifications can be modified.
MIT License
16 stars 14 forks source link

[REFACTOR] :recycle: change routes for users: `markadmin` and `markverified` #167

Open aayushchugh opened 2 years ago

aayushchugh commented 2 years ago

Description

Currently we are using these routes

Mark user as admin --> PATCh /api/admin/users/markadmin/:id Mark user as verified --> PATCH /api/admin/users/markverified/:id Get all users --> GET /api/users Update Username --> PATCh /api/users/:id Delete user --> DELETE /api/admin/users/:id

we can change the routes of markadmin and markverified to follow conventions eg:-

Mark user as admin --> PATCH /api/admin/users/:id/markadmin Mark user as verified --> PATCH /api/admin/users/:id/markverified

Anything else?

No response

aayushchugh commented 2 years ago

@MultiEmail/backend-developers I need feedback that if should we work on this or not

BitOfAByte commented 2 years ago

I can work on this today or tomorrow

is-it-ayush commented 2 years ago

Yeah! They look good. Go ahead with it.