246 endpoint to update user

[itzgeebee]

Pre-Submission Checklist

• Your pull request targets the main branch.
• Branch name is descriptive and follows the guidelines here.
• All tests pass . Use git commit --amend to amend any fixes.

Checklist:

• All tests are passing.
• Tested changes locally.
• Improved documentation or readme where necessary.
• Your commit messages format follows our commit message guideline here.

What does this PR do?

This PR adds the functionality to update users

How should this be manually tested?

1. npm run test

[jocrah]

@beblicarl, please you're needed here as well :)

[beblicarl]

@beblicarl, please you're needed here as well :)

sure

[beblicarl]

@itzgeebee , can you please work on the merge conflict

[itzgeebee]

@itzgeebee , can you please work on the merge conflict

Done @beblicarl

[beblicarl]

@itzgeebee , can you please work on the merge conflict

Done @beblicarl

awesome

[beblicarl]

@itzgeebee , i pulled your branch and there's a lot of errors in your documentation. Kindly use Swagger Viewer extension on vscode so it picks up all those errors for you

[itzgeebee]

@itzgeebee , i pulled your branch and there's a lot of errors in your documentation. Kindly use Swagger Viewer extension on vscode so it picks up all those errors for you

alright I would do that

[beblicarl]

@itzgeebee , i don't think there should be required fields in the schema if you want to update, what do you think?

[itzgeebee]

@itzgeebee , i don't think there should be required fields in the schema if you want to update, what do you think?

It wouldn't work otherwise. Since we are updating multiple fields, it's like we are adding a new user to the databases and if we don't make it required, the not null validation of the db would be triggered and it would throw an error.

[beblicarl]

@itzgeebee , i don't think there should be required fields in the schema if you want to update, what do you think?

It wouldn't work otherwise. Since we are updating multiple fields, it's like we are adding a new user to the databases and if we don't make it required, the not null validation of the db would be triggered and it would throw an error.

okayy

[beblicarl]

@jocrah , i am done, kindly review

[jocrah]

@jocrah , i am done, kindly review

thanks @beblicarl

[itzgeebee]

@jocrah I have made the changes

[jocrah]

@jocrah I have made the changes

@itzgeebee okay nice, taking a look

[jocrah]

nice one @itzgeebee 🚀

[itzgeebee]

nice one @itzgeebee 🚀

Thanks boss @jocrah

[jocrah]

post-merge reviews haha, just documentation